Re: [csswg-drafts] [css-fonts-5] font-size-adjust: ic-height (#8792)

I hope we can resolve this issue soon. This is a blocker for me to implement `ic-height` in Blink and WebKit. While I still prefer to remove `ic-height` from the spec, I am open to other options.

I noticed that the Noto Sans CJK JP font in the WPT repo [1] supports vertical-layout information. Here are some numbers I got from the font with a font-size of 200px:

* Advance Height of “水”: 200px
* Advance Width of “水”: 200px
* Height (Ascent + Descent): 290px
* Ascent: 232px
* Average Character Width: 99.599998px

Given this information, I think the advance width of "水" is the most sensible alternative for 'ic-height' if we decide to keep it in the spec. If that doesn't work, my last preference is a disregard (i.e., no effect) as the fallback behavior.

To sum up, my preference goes in the following order:
1. Removing `ic-height` from the spec
2. Using advance width of “水” as the fallback
3. Disregarding `ic-height` with no effect as the fallback.

[1] /fonts/noto/cjk/NotoSansCJKjp-Regular-subset-chws.otf

-- 
GitHub Notification of comment by shivamidow
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/8792#issuecomment-1855209764 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Thursday, 14 December 2023 06:19:55 UTC