镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

feat(baseball): football and hockey card fonts by default, and room for the live score (1.48.1) - #532

Merged
ChuckBuilds merged 4 commits into
mainfrom
claude/match-scoreboard-fonts
Sep 25, 2026
Merged

ChuckBuilds merged 4 commits into
mainfrom
claude/match-scoreboard-fonts

Conversation

@ChuckBuilds

@ChuckBuilds ChuckBuilds commented Sep 25, 2026 •

Copy link
Copy Markdown
Owner

Why

Baseball's game cards looked different from football's and hockey's in Vegas mode. This PR makes the three scoreboards share one set of default card fonts (users can still set their own), and gives baseball's live card score some room.

What

Before this PR, football and hockey already had identical defaults, and baseball differed in two ways:

  • Status text defaulted to Press Start (size 8 in the schema, 6 in code) instead of 4x6 at 6. Changed in config_schema.json, game_renderer.py and sports.py.
  • Font names: the schema named fonts by alias (press_start, four_by_six) instead of by file name. These are the same fonts, but the three schemas now read the same.

The shared defaults are now:

element font size
score, rank Press Start 2P 10
period, team Press Start 2P 8
status, detail, odds 4x6 6

A font or size a user has set in the web UI is kept. Hockey needed no change.

Live card score: on baseball's live scroll/Vegas card the score sat 1 px under the bases and count, so it read as part of them. It now sits 7 px below them, still clamped above the bottom edge. The two live-card golden images (128×64, 256×128) are regenerated for it; every other golden is unchanged.

Baseball 1.48.1, with a versions[] entry; plugins.json updated. If #523 (baseball 1.49.0) merges first, this needs re-bumping to 1.49.1.

Test plan

  • The three schemas' customization defaults compared field by field: identical.
  • Every scripts/test_*.py guard, check_version_bump.py, check_manifest_version_fields.py, check_sports_drift.py and update_registry.py --check pass against core main.
  • Baseball's plugin tests show the same 4 failing scripts as main under run_plugin_tests.py on Windows. Rerun with UTF-8 console output, three of them pass (test_font_aliases.py included). test_odds_placement.py still fails 2 checks (odds on 64x32 panels), and it fails the same 2 on main, so it's a separate, existing issue.
  • scripts/test_scroll_card_renders.py: all 72 cards match their goldens (Pillow 12.3.0, the version the other 70 were already matching).
  • test_scorebug_element_toggles.py: 32 checks pass.
  • On hdpi, setting baseball's customization to these fonts renders its Vegas cards like football's.

🤖 Generated with Claude Code

…1.48.1)

Status text defaulted to Press Start where football and hockey use 4x6 at 6,
and the schema named its fonts by alias (press_start, four_by_six) where the
other two name the files. The three scoreboards now share identical
defaults for score, period, team, status, detail, rank and odds text.
Anything a user has set keeps their choice.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: ebf334bf-3de9-4f44-817b-ae214e950701


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codacy-production

codacy-production Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity

Metric Results
Complexity 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

ChuckBuilds and others added 3 commits September 25, 2026 11:43
…he count

The score sat one pixel under the bases/count cluster and read as part of
it. Four pixels now, still clamped above the bottom edge.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Only the two live cards change (128x64 and 256x128); every other golden is
untouched. Pillow 12.3.0, the version the other 70 goldens match.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ChuckBuilds ChuckBuilds changed the title feat(baseball): game cards default to the football and hockey fonts (1.48.1) feat(baseball): football and hockey card fonts by default, and room for the live score (1.48.1) Sep 25, 2026
@ChuckBuilds
ChuckBuilds merged commit efc907c into main Sep 25, 2026
4 checks passed
ChuckBuilds added a commit that referenced this pull request Sep 25, 2026
#532 released a 1.48.1 of its own (card fonts, live score spacing). This
branch's unreleased 1.48.1 (update() inside its slot) is folded into its
1.49.0 entry, and main's 1.48.1 sits below it. plugins.json regenerated
from the manifests.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ChuckBuilds
ChuckBuilds deleted the claude/match-scoreboard-fonts branch September 29, 2026 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant