Skip to content

Bundle Twemoji SVG flags and add admin Teams view - #37

Merged
kilgor-trout-c22 merged 5 commits into
masterfrom
fix/flag-emoji-fallback-chrome-issue
Jun 5, 2026
Merged

Bundle Twemoji SVG flags and add admin Teams view#37
kilgor-trout-c22 merged 5 commits into
masterfrom
fix/flag-emoji-fallback-chrome-issue

Conversation

@kilgor-trout-c22

Copy link
Copy Markdown
Contributor

What

Two complementary changes:

Bundled Twemoji SVG flags

National flags no longer depend on the platform's emoji font — they are rendered from SVGs bundled with the module. Fixes flags showing as invisible on Chrome/Linux (regional-indicator pairs unsupported by common system fonts) and inconsistent emoji styles across browsers.

  • 261 flag SVGs from twitter/twemoji v14.0.2 in resources/flags/, served via the existing asset bundle — no CDN involved.
  • Includes the England, Scotland and Wales tag-sequence flags and the Kosovo (XK) flag, building on the TeamNameLocalizer::flagCodepoints() API from Fix flag and name resolution for non-ISO FIFA members (England, Scotland, Wales, Kosovo) #33: the codepoint sequence doubles as the SVG filename, so emoji semantics and asset lookup share one source.
  • License compliance: resources/flags/LICENSE (attribution notice) and LICENSE-GRAPHICS (full CC BY 4.0 text as supplied by the licensor) are included.
  • Mock adapters now use FIFA trigrams for home nations (England ENG, new Scotland SCO replacing Morocco), so the subdivision flags are exercised by test competitions.

Admin Teams view

The competition admin view gains a "Teams" page (analogous to Special bets) listing all participating teams with their flag, name and group — reachable via a "Teams (N)" button. It renders through the shared _team_badge.php partial, so it shows the bundled SVG flags introduced above.

@kilgor-trout-c22
kilgor-trout-c22 merged commit e7d09e6 into master Jun 5, 2026
17 checks passed
@kilgor-trout-c22
kilgor-trout-c22 deleted the fix/flag-emoji-fallback-chrome-issue branch June 5, 2026 15:32
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