Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

font-family is incorrect when lang="zh-Hant" #3834

Closed
carolframpton opened this issue Dec 12, 2022 · 1 comment
Closed

font-family is incorrect when lang="zh-Hant" #3834

carolframpton opened this issue Dec 12, 2022 · 1 comment
Assignees
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@carolframpton
Copy link

🐛 Bug Report

When <html lang="zh-Hant" ... /> the font-family is simplified Chinese not traditional Chinese as expected.

🤔 Expected Behavior

zh-Hant = traditional Chinese

😯 Current Behavior

Spectrum-CSS and RS's copy of Spectrum-CSS do not have an entry for lang:(zh-Hant) so the font family defaults to the one for zh which is the same as zh-Hans which is the wrong font-family.

Currently font-family: 'adobe-clean-han-simplified-c', 'SimSun', 'Heiti SC Light', 'sans-serif';

See https://github.com/adobe/react-spectrum/blob/main/packages/%40adobe/spectrum-css-temp/components/commons/fonts.css#L56

and

https://github.com/adobe/spectrum-css/blob/43a159f9fcc1a5114ea59fd61f5e63ecf6e7b1fa/components/typography/font.css#L29

Note zh-Hant maps to RS's I18nProvider locale="zh-TW".

💁 Possible Solution

Add entry in the above files for zh-Hant.

🔦 Context

Our loc QE spotted this issue and filed a bug against our product. They expect
font-family: 'adobe-clean-han-traditional', 'MingLiu', 'Heiti TC Light','sans-serif';

💻 Code Sample

🌍 Your Environment

Software Version(s)
react-spectrum "3.23.0"
Browser
Operating System

🧢 Your Company/Team

Adobe Admin Console

🕷 Tracking Issue (optional)

@LFDanLu LFDanLu added the bug Something isn't working label Dec 15, 2022
@LFDanLu LFDanLu moved this to ✏️ To Groom in RSP Component Milestones Dec 15, 2022
@LFDanLu LFDanLu added the good first issue Good for newcomers label Dec 15, 2022
@LFDanLu
Copy link
Member

LFDanLu commented Dec 16, 2022

TODO: check that this font is in the typekit

@matthewdeutsch matthewdeutsch moved this from ✏️ To Groom to 📋 Waiting for Sprint in RSP Component Milestones Jan 5, 2023
@ktabors ktabors moved this from 📋 Waiting for Sprint to 🏗 In Progress in RSP Component Milestones Mar 28, 2023
@ktabors ktabors self-assigned this Mar 28, 2023
@devongovett devongovett moved this from 🏗 In Progress to ✅ Done in RSP Component Milestones Apr 10, 2023
@ktabors ktabors closed this as completed Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
Archived in project
Development

No branches or pull requests

3 participants