Skip to content

[css-fonts] Font technology 'incremental' declaration is underspecified #6892

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

Closed
vlevantovsky opened this issue Dec 16, 2021 · 5 comments
Closed
Assignees
Labels

Comments

@vlevantovsky
Copy link

CSS Fonts module was recently amended to add support for font-tech properties, including tech(incremental) in particular. The § 11.1. Font tech describes this as follows:

The incremental tech refers to client support for incremental font loading, using either the range-request or the patch-subset method [PFE-report].

Furthermore, the Example 22 claims to shows how @font-face declaration can be constructed for incremental transfer using the range-request method; however, it is not at all clear whether range-request or a patch-subset methods would be applied.

I believe this font-tech declaration is not sufficient and we should allow authors express their preferences for a particular method of incremental font loading. The data analysis section of the PFE-report (see cost analysis and Figures 17, 20, 23 in particular) makes it very clear that there are major differences between two methods, and the "wrong choice" of incremental font transfer method can have significant adverse effect on font loading time and user experience. Currently, the particular choice of incremental methods is not discussed in the CSS Fonts spec, and it is unclear how user agents would make this determination. The efficacy of incremental loading depends on many factors, including the language of the content and other features - the authors should be able to express their method preferences because the user experience can be significantly affected if the "wrong" incremental method is picked for a specific font in use.

Perhaps the tech(incremental) should be extended to allow tech(incremental-patch) and tech(incremental-range) be declared, in addition to ambivalent tech(incremental) itself.

@svgeesus
Copy link
Contributor

I believe this font-tech declaration is not sufficient and we should allow authors express their preferences for a particular method of incremental font loading.

I agree. All the rest of the tech() (and corresponding font-tech()) express author preferences and constitute client-led content negotiation.

Perhaps the tech(incremental) should be extended to allow tech(incremental-patch) and tech(incremental-range) be declared

I would be fine with those names, which are consistent with the other hyphenated names used there..

@svgeesus svgeesus self-assigned this Jan 18, 2022
@svgeesus
Copy link
Contributor

svgeesus commented Feb 7, 2022

Not hearing any disagreement (but also, no agreement) my plan is to put the two names into the spec and then get review from Web Fonts WG

@litherum
Copy link
Contributor

litherum commented Mar 8, 2022

This is being discussed w3c/IFT#57. Let's let that conversation come to a conclusion before making any edits.

@svgeesus
Copy link
Contributor

svgeesus commented Mar 8, 2022

Fair enough, since that issue does not seem close to consensus

@svgeesus
Copy link
Contributor

svgeesus commented Mar 1, 2023

Closed by a951f1f

@svgeesus svgeesus closed this as completed Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants