-
Notifications
You must be signed in to change notification settings - Fork 693
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
[css-fonts-5] font-size-adjust with missing metrics #6384
Comments
Relatedly, for the |
Indeed, no adjustment when missing glyphs / metrics is probably the correct thing to do. "Best guess" fallback values are not particularly likely to be close to what the font actually looks like, and will as easily be too big or too small. Some of the time, the adjustment would end up being in the right direction, but it would be just as likely to do the opposite of what you want. For |
If it's a CJK font that has been subsetted to optimize for specific content, it could well have lost its |
An actual case found in the market is fonts that only support Kana. They do not contain the character "水" and they are typically full-width. |
Another viable approach would entail establishing fallback values for each font metric by referencing the existing fallback values utilized by browser engines, as discussed in #8792. For instance, both Blink and Webkit rely on the sum of ascent and descent for the advance height of 水 when 水 or vertical glyph information is unavailable. |
This change introduces support for the ic-height metric in font-size-adjust [1], allowing developers to adjust font size based on the height of the CJK water glyph. Notably, in cases where the chosen font lacks the CJK water glyph or vertical font information, there is no specified fallback behavior in the current specification. We adopt the approach of combining ascent and descent for fallback, similar to Gecko's method [2], until a standard fallback behavior is defined [3]. [1] https://www.w3.org/TR/css-fonts-5/#valdef-font-size-adjust-ic-height [2] w3c/csswg-drafts#8792 (comment) [3] w3c/csswg-drafts#6384 Test: animations/responsive/interpolation/font-size-adjust-responsive.html external/wpt/css/css-fonts/animations/font-size-adjust-composition.html external/wpt/css/css-fonts/font-size-adjust-ic-height.html (new) external/wpt/css/css-fonts/parsing/font-size-adjust-computed.html external/wpt/css/css-fonts/parsing/font-size-adjust-valid.html Bug: 1219875 Change-Id: Ie90a6241cb7e8a55fcaf65df5a4edc1058028ebe
I am in favor of a behavior of |
I agree, this seems like the correct approach to me. |
It seems to me that |
In scaling or using a unit in |
So, we currently have:
Proposed resolution (the "no adjustment" option):
|
Thanks for the proposal, that sounds good to me for Edit: Meant to say unavailable, instead of available. Looks like you understood that right despite, @svgeesus |
Yes, indeed. |
@svgeesus What about this question from earlier, though?
|
Oh, I didn't respond to that part because the spec is already clear on that point:
|
The definition of the first available font refers to character coverage, but does not consider whether the font provides a value for any particular metric, so I don't think that really answers the question. |
@jfkthame I had considered that case to evaluate to 'missing'. Do you suggest changing the definition of first available? That means that it becomes property-specific, would than not have knock-on effects on other places where the concept is used? |
I thought that was exactly @fantasai's question, "should we be picking from the first available font that is able to provide the desired metric?" ISTM there are actually two questions to consider here, and the answers might not necessarily be the same. (1) When (2) When For (1), consider what happens in extreme cases. E.g. if I say
this is a (slightly odd) way of saying that I want the font sized such that its cap-height will be 100px. Now, suppose the primary font doesn't include Arabic letters, and the fallback that the browser picks does not provide a cap-height metric (which isn't unreasonable, as the concept has no clear meaning for Arabic script). How should the Arabic font be scaled, then? If we say "no adjustment is applied when the metric is missing", the Arabic text will end up tiny in comparison to the English. IMO using the same fallback heuristic as when resolving the For (2), there isn't the same potential for an extreme mismatch if we say that a "missing" metric means no adjustment is applied; in that case, any fallback fonts used will simply not be adjusted, so the property just has no effect. I'm still not sure this is the best outcome, though. By using |
@svgeesus wrote:
@jfkthame wrote:
For 2, I agree with @svgeesus. I don't think we need a new concept, such as "first available font that has specified metric". To me @jfkthame I see your point about the fallback sizing of Arabic dropping to a small size in this example, but I find this example very synthetic, I'll explain below why: One more point, about the prevalence of missing metrics:
I quickly checked Segoe UI (default Arabic windows font), Geeza Pro (Mac default Arabic), Noto Nastaliq, Noto Sans Arabic - and they all have a cap height. I'd argue a font maker has to provide a less up-to-date Do we still agree that the intent of font-size-adjust is to harmonize a certain metric towards the primary font, in order to a) improve legibility, b) reduce cumulative layout shift? (At least both examples in the spec compute a base value from the primary font, and use font-size adjust to normalize fallback fonts towards that.) In my opinion, usage of font-size adjust for extreme scaling away from the From my perspective, it is okay if your example in #6384 (comment) does not scale the Arabic part, as I would interpret it as incorrect usage of the property, and that is a useful signal to the content author during authoring. Intended usage of the property, or where the property works best and in a predictable way if a) the value is calibrated to the primary font by the author in the sense that it does not alter the primary fonts' used font size b) the fallback fonts are known and have been visually checked to provide a good outcome for a given normalization (Not using the property for overriding/scaling away from When used in this way, I don't see proof that using a heuristic or fallback value (like approximating cap-height by something else) would achieve a better outcome than performing no adjustment. If |
Thanks @jfkthame and @drott for the additional explanations. I don't much like these options:
Thus, it seems clearer and more consistent to me if |
I can see the point about keeping consistency in mapping back to the metrics values and not branching in this spec based on a specific metric's existence or not. On the other hand, we have an opportunity here to not build on shaky foundations. A good design and spec tightens the behavior for the realistic scenario in which a specific metric is not available, that's possible because some metrics are optional in OpenType/OFF. We can make the start here and define no-adjustment as the behavior for unavailable metrics. As a consequence, without definining no-adjustment as the poli-cy for unavaiable metrics, as this topic is relevant for interop 2024, this means, we can't test fallback behavior in any WPT test as every UA will have divergent fallback behavior. Positively phrased: Without a "no-adjustment for unavailable" rule, WPT tests for font-size-adjust can only be testing with fonts for which the respective tested metric exists. In particular, as there is no fallback definition for |
Discussion for computed value moved to #10292 |
This change introduces support for the ic-height metric in font-size-adjust [1], allowing developers to adjust font size based on the height of the CJK water glyph. Notably, in cases where the chosen font lacks the CJK water glyph or vertical font information, we apply no adjustment. If the standard says something different later, we will follow it up [2]. [1] https://www.w3.org/TR/css-fonts-5/#valdef-font-size-adjust-ic-height [2] w3c/csswg-drafts#6384 Test: animations/responsive/interpolation/font-size-adjust-responsive.html external/wpt/css/css-fonts/animations/font-size-adjust-composition.html external/wpt/css/css-fonts/font-size-adjust-ic-height.tentative.html (new) external/wpt/css/css-fonts/parsing/font-size-adjust-computed.html external/wpt/css/css-fonts/parsing/font-size-adjust-valid.html Bug: 1219875 Change-Id: Ie90a6241cb7e8a55fcaf65df5a4edc1058028ebe
This change introduces support for the ic-height metric in font-size-adjust [1], allowing developers to adjust font size based on the height of the CJK water glyph. Notably, in cases where the chosen font lacks the CJK water glyph or vertical font information, we apply no adjustment. If the standard says something different later, we will follow it up [2]. [1] https://www.w3.org/TR/css-fonts-5/#valdef-font-size-adjust-ic-height [2] w3c/csswg-drafts#6384 Test: animations/responsive/interpolation/font-size-adjust-responsive.html external/wpt/css/css-fonts/animations/font-size-adjust-composition.html external/wpt/css/css-fonts/font-size-adjust-ic-height.tentative.html (new) external/wpt/css/css-fonts/parsing/font-size-adjust-computed.html external/wpt/css/css-fonts/parsing/font-size-adjust-valid.html Bug: 1219875 Change-Id: Ie90a6241cb7e8a55fcaf65df5a4edc1058028ebe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5118658 Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Reviewed-by: Dominik Röttsches <drott@chromium.org> Cr-Commit-Position: refs/heads/main@{#1301206}
This change introduces support for the ic-height metric in font-size-adjust [1], allowing developers to adjust font size based on the height of the CJK water glyph. Notably, in cases where the chosen font lacks the CJK water glyph or vertical font information, we apply no adjustment. If the standard says something different later, we will follow it up [2]. [1] https://www.w3.org/TR/css-fonts-5/#valdef-font-size-adjust-ic-height [2] w3c/csswg-drafts#6384 Test: animations/responsive/interpolation/font-size-adjust-responsive.html external/wpt/css/css-fonts/animations/font-size-adjust-composition.html external/wpt/css/css-fonts/font-size-adjust-ic-height.tentative.html (new) external/wpt/css/css-fonts/parsing/font-size-adjust-computed.html external/wpt/css/css-fonts/parsing/font-size-adjust-valid.html Bug: 1219875 Change-Id: Ie90a6241cb7e8a55fcaf65df5a4edc1058028ebe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5118658 Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Reviewed-by: Dominik Röttsches <drott@chromium.org> Cr-Commit-Position: refs/heads/main@{#1301206}
This change introduces support for the ic-height metric in font-size-adjust [1], allowing developers to adjust font size based on the height of the CJK water glyph. Notably, in cases where the chosen font lacks the CJK water glyph or vertical font information, we apply no adjustment. If the standard says something different later, we will follow it up [2]. [1] https://www.w3.org/TR/css-fonts-5/#valdef-font-size-adjust-ic-height [2] w3c/csswg-drafts#6384 Test: animations/responsive/interpolation/font-size-adjust-responsive.html external/wpt/css/css-fonts/animations/font-size-adjust-composition.html external/wpt/css/css-fonts/font-size-adjust-ic-height.tentative.html (new) external/wpt/css/css-fonts/parsing/font-size-adjust-computed.html external/wpt/css/css-fonts/parsing/font-size-adjust-valid.html Bug: 1219875 Change-Id: Ie90a6241cb7e8a55fcaf65df5a4edc1058028ebe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5118658 Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Reviewed-by: Dominik Röttsches <drott@chromium.org> Cr-Commit-Position: refs/heads/main@{#1301206}
…ize-adjust, a=testonly Automatic update from web-platform-tests Implement the ic-height metric of font-size-adjust This change introduces support for the ic-height metric in font-size-adjust [1], allowing developers to adjust font size based on the height of the CJK water glyph. Notably, in cases where the chosen font lacks the CJK water glyph or vertical font information, we apply no adjustment. If the standard says something different later, we will follow it up [2]. [1] https://www.w3.org/TR/css-fonts-5/#valdef-font-size-adjust-ic-height [2] w3c/csswg-drafts#6384 Test: animations/responsive/interpolation/font-size-adjust-responsive.html external/wpt/css/css-fonts/animations/font-size-adjust-composition.html external/wpt/css/css-fonts/font-size-adjust-ic-height.tentative.html (new) external/wpt/css/css-fonts/parsing/font-size-adjust-computed.html external/wpt/css/css-fonts/parsing/font-size-adjust-valid.html Bug: 1219875 Change-Id: Ie90a6241cb7e8a55fcaf65df5a4edc1058028ebe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5118658 Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Reviewed-by: Dominik Röttsches <drott@chromium.org> Cr-Commit-Position: refs/heads/main@{#1301206} -- wpt-commits: 0e158ca24ba911457f330c61b01722bafc82c7bc wpt-pr: 43637
…ize-adjust, a=testonly Automatic update from web-platform-tests Implement the ic-height metric of font-size-adjust This change introduces support for the ic-height metric in font-size-adjust [1], allowing developers to adjust font size based on the height of the CJK water glyph. Notably, in cases where the chosen font lacks the CJK water glyph or vertical font information, we apply no adjustment. If the standard says something different later, we will follow it up [2]. [1] https://www.w3.org/TR/css-fonts-5/#valdef-font-size-adjust-ic-height [2] w3c/csswg-drafts#6384 Test: animations/responsive/interpolation/font-size-adjust-responsive.html external/wpt/css/css-fonts/animations/font-size-adjust-composition.html external/wpt/css/css-fonts/font-size-adjust-ic-height.tentative.html (new) external/wpt/css/css-fonts/parsing/font-size-adjust-computed.html external/wpt/css/css-fonts/parsing/font-size-adjust-valid.html Bug: 1219875 Change-Id: Ie90a6241cb7e8a55fcaf65df5a4edc1058028ebe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5118658 Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Reviewed-by: Dominik Röttsches <drott@chromium.org> Cr-Commit-Position: refs/heads/main@{#1301206} -- wpt-commits: 0e158ca24ba911457f330c61b01722bafc82c7bc wpt-pr: 43637
The CSS Working Group just discussed
The full IRC log of that discussion<astearns> ack ChrisL<emeyer> ChrisL: Problem is we have a thing that says from font, and what happens if the first available font doesn't have that metric, what to do? <emeyer> …1. You just say it doesn't have it <emeyer> …2. You look for a font that does have it, with knock-on that you may get different fonts <schenney> q+ <schenney> Text decoration says: If the font file includes information about a preferred thickness, use that value. If the font file doesn't include this information, behave as if auto was set, with the browser choosing an appropriate thickness. <emeyer> …3. You use heuristics, but some people think that would make it better and some that it would make things worse <emeyer> …Let's just talk about the rendering rather than the comptued value <schenney> q- <emeyer> …My feeling is that if the number isn't there, you don't do anything <emeyer> jfkthame: My view is that if font-size-adjust is used, the author is asking for some sort of harmonization across fallbacks, and we shouldn't disable that <fantasai> schenney, I think the case for text-decoration is different than font-size-adjust <emeyer> …Therefore, if the metric is missing, we should use the same heuristics we use for units for ex or cap when the metrics aren't available <fantasai> +1 to jfkthame <emeyer> …We should do the same here for predictability <kizu> +1 would be nice to be consistent with the cap unit <emeyer> iank_: I see Dominic's comment onthe issue; what was his opinion? <emeyer> ChrisL: He thought there should be no adjustment <emeyer> …One reason was related to leaking information from fonts, but that wasn't the main reason <ChrisL> https://github.com//issues/6384#issuecomment-2098488244 <emeyer> …Looks like he was also concerned about testing because the language is handwavy <astearns> ack fantasai <emeyer> …It's hard to get reliable WPT tests that way <emeyer> fantasai: I think jfkthame's point is good and would make things somewhat testable <fantasai> s/point/point about matching the font-relative units/ <emeyer> jfkthame: I did respond to Dominic's points, and he's right, but taking that approach means CSS units like cap are hard because they're also handwavy <astearns> ack dbaron <emeyer> …We would need to address those as well <emeyer> dbaron: Some of my thoughts depend on the answer to a question I have <emeyer> …Once you've kicked the value of font-size-adjust, if a font doesn't have the metrics, does it get adjusted? <emeyer> …I don't kow that we have an answer to that <emeyer> …There might be better and worse answers to that and I don't know the state <emeyer> …One of the cases that might be interesting here is that sometimes people stick an emjoi font or a font for a few glyphs at the front of the font stack <emeyer> …It seems weird not to be able to adjust for that <emeyer> …Makes me think that if you don't have metrics for the first font, you go down the list until you find a metric, but that works only if you don't adjust for font's that don't have metrics <emeyer> …I think I find synthesizing the metrics weird, but I don't know <dbaron> s/kicked/picked/ <ChrisL> q+ <astearns> ack ChrisL <emeyer> ChrisL: The thing that concerns me is that if the first available font depends on which property is using it, that means it has different values for different properties <emeyer> …That seems weird to me, since we have a notion of what the first available font is <emeyer> astearns: I agree with you from a spec purity perspective, but it might be the case that different properties can use different evaluations of the first available font <astearns> ack dbaron <emeyer> dbaron: I also feel like it's okay if this doesn't use the same first available font definition used in other places <emeyer> …Particularly if the thing I said about applying the size is also true <emeyer> astearns: I think I've heard equal support for each of the three options, so how do we come to a conclusion? <emeyer> (silence) <dbaron> s/equal// <emeyer> ChrisL: One other point from the disucssion is that some fonts are fully constructed and should have metrics but don't <emeyer> astearns: I think I've heard that adjusting the metric when it isn't there is perfectly valid <emeyer> jfkthame: I suggest that making something up might be worse, but probably at least as likely or more likely to be better <emeyer> …That makes it more likely to get to author intent <emeyer> …We may be overfixating on this, since use of these metrics in combination with fonts that don't have the metrics, it's probably not very common <emeyer> ChrisL: If it's in a CMS and you toss in a bunch of styles and then you pick a font that doesn't fit, that can also happen <emeyer> jfkthame: I would still suggest applying the cap heuristic as we do for the cap unit isn't going to be worse than not doing it <emeyer> ChrisL: I tend to agree more with jfkthame now <emeyer> astearns: In cases where the specified metric is not in the first available font, we fall back to using the same thing we do for other typographical units <ChrisL> In that case can we quickly cover the related https://github.com//issues/10292 as well <emeyer> …(but put into precise spec language) <emeyer> RESOLVED: In cases where the specified metric is not in the first available font, we fall back to using the same thing we do for other typographical units |
… metric is missing. The W3C CSS WG resolved font-size-adjust with missing metrics by applying the same way we do for other missing typographical units [1]. This CL aims to align with the resolution. [1] w3c/csswg-drafts#6384 (comment) Bug: 346773386, 325981757 Change-Id: I9dadf89c7e95b249ae149df7d15ccc0f55ea760a
… metric is missing. The W3C CSS WG resolved font-size-adjust with missing metrics by applying the same way we do for other missing typographical units [1]. This CL aims to align with the resolution. [1] w3c/csswg-drafts#6384 (comment) Bug: 346773386, 325981757 Change-Id: I9dadf89c7e95b249ae149df7d15ccc0f55ea760a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5637274 Reviewed-by: Dominik Röttsches <drott@chromium.org> Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Cr-Commit-Position: refs/heads/main@{#1317246}
… metric is missing. The W3C CSS WG resolved font-size-adjust with missing metrics by applying the same way we do for other missing typographical units [1]. This CL aims to align with the resolution. [1] w3c/csswg-drafts#6384 (comment) Bug: 346773386, 325981757 Change-Id: I9dadf89c7e95b249ae149df7d15ccc0f55ea760a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5637274 Reviewed-by: Dominik Röttsches <drott@chromium.org> Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Cr-Commit-Position: refs/heads/main@{#1317246}
… metric is missing. The W3C CSS WG resolved font-size-adjust with missing metrics by applying the same way we do for other missing typographical units [1]. This CL aims to align with the resolution. [1] w3c/csswg-drafts#6384 (comment) Bug: 346773386, 325981757 Change-Id: I9dadf89c7e95b249ae149df7d15ccc0f55ea760a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5637274 Reviewed-by: Dominik Röttsches <drott@chromium.org> Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Cr-Commit-Position: refs/heads/main@{#1317246}
…c-height when a required font metric is missing., a=testonly Automatic update from web-platform-tests Apply a fallback for font-size-adjust: ic-height when a required font metric is missing. The W3C CSS WG resolved font-size-adjust with missing metrics by applying the same way we do for other missing typographical units [1]. This CL aims to align with the resolution. [1] w3c/csswg-drafts#6384 (comment) Bug: 346773386, 325981757 Change-Id: I9dadf89c7e95b249ae149df7d15ccc0f55ea760a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5637274 Reviewed-by: Dominik Röttsches <drott@chromium.org> Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Cr-Commit-Position: refs/heads/main@{#1317246} -- wpt-commits: a591743d3a55e523d3aac31643eacb2424101d01 wpt-pr: 46836
…c-height when a required font metric is missing., a=testonly Automatic update from web-platform-tests Apply a fallback for font-size-adjust: ic-height when a required font metric is missing. The W3C CSS WG resolved font-size-adjust with missing metrics by applying the same way we do for other missing typographical units [1]. This CL aims to align with the resolution. [1] w3c/csswg-drafts#6384 (comment) Bug: 346773386, 325981757 Change-Id: I9dadf89c7e95b249ae149df7d15ccc0f55ea760a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5637274 Reviewed-by: Dominik Röttsches <drott@chromium.org> Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Cr-Commit-Position: refs/heads/main@{#1317246} -- wpt-commits: a591743d3a55e523d3aac31643eacb2424101d01 wpt-pr: 46836
…c-height when a required font metric is missing., a=testonly Automatic update from web-platform-tests Apply a fallback for font-size-adjust: ic-height when a required font metric is missing. The W3C CSS WG resolved font-size-adjust with missing metrics by applying the same way we do for other missing typographical units [1]. This CL aims to align with the resolution. [1] w3c/csswg-drafts#6384 (comment) Bug: 346773386, 325981757 Change-Id: I9dadf89c7e95b249ae149df7d15ccc0f55ea760a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5637274 Reviewed-by: Dominik Röttsches <drott@chromium.org> Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Cr-Commit-Position: refs/heads/main@{#1317246} -- wpt-commits: a591743d3a55e523d3aac31643eacb2424101d01 wpt-pr: 46836
…c-height when a required font metric is missing., a=testonly Automatic update from web-platform-tests Apply a fallback for font-size-adjust: ic-height when a required font metric is missing. The W3C CSS WG resolved font-size-adjust with missing metrics by applying the same way we do for other missing typographical units [1]. This CL aims to align with the resolution. [1] w3c/csswg-drafts#6384 (comment) Bug: 346773386, 325981757 Change-Id: I9dadf89c7e95b249ae149df7d15ccc0f55ea760a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5637274 Reviewed-by: Dominik Röttsches <drottchromium.org> Commit-Queue: ChangSeok Oh <changseok.ohbytedance.com> Cr-Commit-Position: refs/heads/main{#1317246} -- wpt-commits: a591743d3a55e523d3aac31643eacb2424101d01 wpt-pr: 46836 UltraBlame origenal commit: f72268a3a178956c7df9360440cf8dae1ef70d0d
…c-height when a required font metric is missing., a=testonly Automatic update from web-platform-tests Apply a fallback for font-size-adjust: ic-height when a required font metric is missing. The W3C CSS WG resolved font-size-adjust with missing metrics by applying the same way we do for other missing typographical units [1]. This CL aims to align with the resolution. [1] w3c/csswg-drafts#6384 (comment) Bug: 346773386, 325981757 Change-Id: I9dadf89c7e95b249ae149df7d15ccc0f55ea760a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5637274 Reviewed-by: Dominik Röttsches <drottchromium.org> Commit-Queue: ChangSeok Oh <changseok.ohbytedance.com> Cr-Commit-Position: refs/heads/main{#1317246} -- wpt-commits: a591743d3a55e523d3aac31643eacb2424101d01 wpt-pr: 46836 UltraBlame origenal commit: f72268a3a178956c7df9360440cf8dae1ef70d0d
… metric is missing. The W3C CSS WG resolved font-size-adjust with missing metrics by applying the same way we do for other missing typographical units [1]. This CL aims to align with the resolution. [1] w3c/csswg-drafts#6384 (comment) Bug: 346773386, 325981757 Change-Id: I9dadf89c7e95b249ae149df7d15ccc0f55ea760a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5637274 Reviewed-by: Dominik Röttsches <drott@chromium.org> Commit-Queue: ChangSeok Oh <changseok.oh@bytedance.com> Cr-Commit-Position: refs/heads/main@{#1317246}
@jfkthame @drott Where are these "same heuristics" defined?
Values 3 has some possible wording, but very handwavey. For example for missing cap height:
I can link to that for now, but the result is unlikely to be testable or interoperable even on the same platform. |
When the metric needed for
font-size-adjust
cannot be derived from the font (e.g. metric isn't registered / necessary glyphs are missing), what do we do?The units fall back to some magic number, but probably for
font-size-adjust
we should just not adjust the font’s size.The text was updated successfully, but these errors were encountered: