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

ADOBE font fallback is incorrect for Korean / Chinese fonts. #232

Closed
yloubry opened this issue Aug 2, 2019 · 2 comments · Fixed by #265
Closed

ADOBE font fallback is incorrect for Korean / Chinese fonts. #232

yloubry opened this issue Aug 2, 2019 · 2 comments · Fixed by #265
Assignees
Labels
bug Results from a bug in the CSS implementation

Comments

@yloubry
Copy link

yloubry commented Aug 2, 2019

Expected Behavior

For Adobe Clean fonts

 --spectrum-font-family-zh: var(--spectrum-font-family-han);
 --spectrum-font-family-zhhans: var(--spectrum-font-family-han);
 --spectrum-font-family-ko: var(--spectrum-font-family-han);
 --spectrum-font-family-ja: var(--spectrum-font-family-han);

https://github.com/adobe/spectrum-css/blame/03952ed37e88d2b49802414a8d0e098a3153e582/src/commons/fonts.css#L29

should point to their respective fallback families.

[lang=ja] & {
    body {
      font-family: 'adobe-clean-han-japanese', 'Yu Gothic', '\30E1 \30A4 \30EA \30AA', '\30D2 \30E9 \30AE \30CE \89D2 \30B4  Pro W3', 'Hiragino Kaku Gothic Pro W3', 'Osaka', '\FF2D \FF33 \FF30 \30B4 \30B7 \30C3 \30AF', 'MS PGothic', 'sans-serif';
    }
}
[lang=zh-Hant] & {
    body {
      font-family: 'adobe-clean-han-traditional', 'MingLiu', 'Heiti TC Light','sans-serif';
    }
}
[lang=zh-Hans] & {
    body {
      font-family: 'adobe-clean-han-simplified-c', 'SimSun', 'Heiti SC Light', 'sans-serif';
    }
}
[lang=ko] & {
   body {
      font-family: 'adobe-clean-han-korean', 'Malgun Gothic', 'Apple Gothic', 'sans-serif';
   }
}

Actual Behavior

Currently, all CJK languages fallsback to Japanese Han fonts subset.

Reproduce Scenario (including but not limited to)

Steps to Reproduce

Browser name/version/os (ie Chrome Version 62.0.3202.94 (Official Build) (64-bit) MacOS)

Spectrum-CSS version

Sample Code that illustrates the problem

Screenshots (if applicable)

@yloubry
Copy link
Author

yloubry commented Aug 2, 2019

Contact yloubry on company slack for more info.

@lazd lazd added Component: Typography bug Results from a bug in the CSS implementation labels Aug 8, 2019
@adobe-bot
Copy link

JIRA issue created: https://jira.corp.adobe.com/browse/SDS-1956

@lazd lazd mentioned this issue Aug 19, 2019
@lazd lazd self-assigned this Aug 20, 2019
lazd added a commit that referenced this issue Aug 22, 2019
* Add Adobe Clean UX support, closes #189 
* Fix fallback fonts for ar, he, ja, zh-Hans, zh-Hant, and ko, closes #232
* Change zhhant fallbacks per @tappily
@lazd lazd closed this as completed in 06688fa Aug 23, 2019
jianliao pushed a commit that referenced this issue Aug 27, 2019
* New, responsive site based on individually versioned components with silly fast navigation, lazy loaded deps as docs are viewed
  * Basic accessible search with build-time index generation
* Update README for individual versioning, add legacy README with legacy usage
  * Add documentation for site updating and architecture
  * Add docs for documentation generation, link to docs, remove references to Precursors
  * Add instructions for adding/editing components
  * Add docs to show how to update DNA and icons
  * Add docs for getting DNA variables from @spectrum-css/vars
  * Document issues with npm ci and installing optional dependencies

* Update to DNA 5.18.1
* Update icons to 4.1.0
* Audit DNA status for all components
* Add nvmrc
* Export a experimental tree-shakable DNA variable object from @spectrum-css/vars

* Update Textarea line-height and min-height, closes #231
* Fix spinner buttons appearing on Steppers in Firefox, closes #214
* Fix search field on iOS, closes #229 
* Fix Menu chevron vertical alignment, fixes #240
* Fix horizontal alignment of labels in anchor buttons, fixes #239
* Remove button outline when focused in Firefox, fixes #161
* Fix bar loader label in IE 11, fixes #242
* Fix Radio label margins when labels are below, fixes #246
* Fix Checkbox icon color when checkbox imported before icons, fixes #219 (#235)
* Add Adobe Clean UX support, closes #189  (#248)
* Fix fallback fonts for ar, he, ja, zh-Hans, zh-Hant, and ko, closes #232 (#248)
* Removed float from tags, fixes #218 (#237)
* Make Slider grab handle look right in docs, closes #255
* Add missing Asset docs, closes #256
* Add Menu .is-highlighted so we can indicate highlight without mis-using .is-open, closes #258 
* Support using links as menu items, closes #257
* Fix incorrect height for small BarLoader, fixes #259
jianliao pushed a commit that referenced this issue Sep 19, 2019
Update Textarea line-height and min-height, closes #231
Fix spinner buttons appearing on Steppers in Firefox, closes #214
Fix search field on iOS, closes #229
Fix Menu chevron vertical alignment, fixes #240
Fix horizontal alignment of labels in anchor buttons, fixes #239
Remove button outline when focused in Firefox, fixes #161
Fix bar loader label in IE 11, fixes #242
Fix Radio label margins when labels are below, fixes #246
Fix Checkbox icon color when checkbox imported before icons, fixes #219 (#235)
Add Adobe Clean UX support, closes #189 (#248)
Fix fallback fonts for ar, he, ja, zh-Hans, zh-Hant, and ko, closes #232 (#248)
Removed float from tags, fixes #218 (#237)
Make Slider grab handle look right in docs, closes #255
Add missing Asset docs, closes #256
Add Menu .is-highlighted so we can indicate highlight without mis-using .is-open, closes #258
Support using links as menu items, closes #257
Fix incorrect height for small BarLoader, fixes #259
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Results from a bug in the CSS implementation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants