Skip to content

Rename "Symbol"-named embedded fonts to avoid Windows substitution#21326

Draft
calixteman wants to merge 1 commit into
mozilla:masterfrom
calixteman:issue21303
Draft

Rename "Symbol"-named embedded fonts to avoid Windows substitution#21326
calixteman wants to merge 1 commit into
mozilla:masterfrom
calixteman:issue21303

Conversation

@calixteman
Copy link
Copy Markdown
Contributor

@calixteman calixteman commented May 24, 2026

Edge on Windows substitutes the system Symbol font for any embedded font
whose name table identifies it as "Symbol"/"SymbolMT", ignoring the
embedded glyphs and mis-rendering e.g. the bullet character as Greek xi.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.42%. Comparing base (1cb14a0) to head (c9cb51c).
⚠️ Report is 18 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #21326   +/-   ##
=======================================
  Coverage   81.42%   81.42%           
=======================================
  Files         257      257           
  Lines       65351    65357    +6     
=======================================
+ Hits        53209    53218    +9     
+ Misses      12142    12139    -3     
Flag Coverage Δ
fonttest 9.11% <50.00%> (+<0.01%) ⬆️
integrationtest 67.00% <100.00%> (+<0.01%) ⬆️
unittest 56.87% <50.00%> (+<0.01%) ⬆️
unittestcli 56.06% <50.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@calixteman
Copy link
Copy Markdown
Contributor Author

It's an attempt to fix #21303.
I can't reproduce locally, so @janekh, if you could test in the viewer I'll generate, it'd be nice.

/botio-linux preview

@moz-tools-bot
Copy link
Copy Markdown
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_preview from @calixteman received. Current queue size: 0

Live output at: http://54.241.84.105:8877/a5c452ddf6581a0/output.txt

@moz-tools-bot
Copy link
Copy Markdown
Collaborator

From: Bot.io (Linux m4)


Success

Full output at http://54.241.84.105:8877/a5c452ddf6581a0/output.txt

Total script time: 1.19 mins

Published

@janekh
Copy link
Copy Markdown

janekh commented May 25, 2026

Unfortunately, this one PC still displays the bullets wrong.

Edge on Windows substitutes the system Symbol font for any embedded font
whose name table identifies it as "Symbol"/"SymbolMT", ignoring the
embedded glyphs and mis-rendering e.g. the bullet character as Greek xi.
@calixteman calixteman changed the title Clear OS/2 ulCodePageRange1 Symbol bit when validating embedded fonts Rename "Symbol"-named embedded fonts to avoid Windows substitution May 25, 2026
@calixteman
Copy link
Copy Markdown
Contributor Author

/botio-linux preview

@moz-tools-bot
Copy link
Copy Markdown
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_preview from @calixteman received. Current queue size: 0

Live output at: http://54.241.84.105:8877/5255318da14fde2/output.txt

@moz-tools-bot
Copy link
Copy Markdown
Collaborator

From: Bot.io (Linux m4)


Success

Full output at http://54.241.84.105:8877/5255318da14fde2/output.txt

Total script time: 1.18 mins

Published

@calixteman
Copy link
Copy Markdown
Contributor Author

calixteman commented May 25, 2026

@janekh can you try with the viewer I've just created ?

@janekh
Copy link
Copy Markdown

janekh commented May 26, 2026

Still no change. Is there anything I can check on this specific PC to help track down this thing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants