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

Does Client Hints have multi vendor support? #867

Closed
othermaciej opened this issue Feb 3, 2019 · 3 comments
Closed

Does Client Hints have multi vendor support? #867

othermaciej opened this issue Feb 3, 2019 · 3 comments
Labels
removal/deprecation Removing or deprecating a feature topic: client hints

Comments

@othermaciej
Copy link

The Fetch specification integrates support for Client Hints, most notably in step 7 of the Fetching algorithm.

Per WHATWG Working Mode, additions are supposed to have "support from at least two implementors", and "the support from implementers should be of the form 'we would like to implement this soon' and not just 'this seems like a reasonable idea'".

I know that Chrome/Blinks supports (and ships) Client Hints and Safari/WebKit does not ship Clients Hints or support it. Mozilla's standards position on the topic doesn't hard refuse, but also doesn't seem to indicate any kind of active support. Are there indications of multi-implementor support that I am missing?

I don't really want to be policing this, but since features shipping only in Safari have been removed from specs based on the multi-implementor support policy before, I think it should be applied consistently. There are also currently open issues for further extending this support, and I'm concerned they may give the impression that Client Hints is part of the interoperable multi-vendor Web Platform.

@yoavweiss
Copy link
Collaborator

@othermaciej AFAIK, the current Client Hints language was added to the spec before that rule was in place. Furthermore, the current language doesn't represent the current Chromium implementation, nor where we want to take it. The latter is defined in a WIP PR, until we're able to land it as the feature gets multi-vendor support.

@othermaciej
Copy link
Author

AFAIK, the current Client Hints language was added to the spec before that rule was in place.

Looks like it was added in this commit on March 22, 2016: 404dc3a

Working Mode was first explicitly made a separate document in April 25th, 2017. So it seems you are correct about timing.

It's not clear to me whether the policy is supposed to apply retroactively or if so in what cases. However, there have been subsequent updates, the most recent in 2018, still without a showing of multi-implementor support.

If the current language does not have multi-implementor support, and also does not match what the one existing implementation does, perhaps it should be removed in the meantime?

I leave it to the editor's discretion to determine the best course of action here.

@annevk annevk added removal/deprecation Removing or deprecating a feature topic: client hints labels Feb 4, 2019
@annevk
Copy link
Member

annevk commented Feb 4, 2019

I've been thinking about this as well, especially as the model as-is is broken and might confuse potential implementers.

I'd be willing to create a PR to remove it and the resulting commit could be used by @yoavweiss in the PR to eventually add it again, provided support materializes.

annevk added a commit that referenced this issue Mar 23, 2019
The model described does not match implementations and there's only support from a single implementation at this point.

#726 works on reintroducing this feature with broader support and a better model.

Closes #867.
annevk added a commit that referenced this issue Mar 23, 2019
The model described does not match implementations and there's only support from a single implementation at this point.

#726 works on reintroducing this feature with broader support and a better model.

Closes #867.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
removal/deprecation Removing or deprecating a feature topic: client hints
Development

No branches or pull requests

3 participants