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

Drop non-normative references to execCommand() #7064

Conversation

sideshowbarker
Copy link
Contributor

@sideshowbarker sideshowbarker commented Sep 13, 2021

https://w3c.github.io/editing/docs/execCommand/ has a big red warning:

This spec is incomplete and it is not expected that it will advance beyond draft status. Authors should not use most of these features directly, but instead use JavaScript editing libraries. The features described in this document are not implemented consistently or fully by user agents, and it is not expected that this will change in the foreseeable future.

Given all that, it seems not good to have text in the HTML spec saying, “User agents are encouraged to implement the features described in execCommand”, and also not good to make reference to the execCommand() API as something that developers should be using directly in their code.


/infrastructure.html ( diff )
/interaction.html ( diff )

https://w3c.github.io/editing/docs/execCommand/ has a big red warning:

> This spec is incomplete and it is not expected that it will advance
> beyond draft status. Authors should not use most of these features
> directly, but instead use JavaScript editing libraries. The features
> described in this document are not implemented consistently or fully
> by user agents, and it is not expected that this will change in the
> foreseeable future.

Given all that, it seems not good to have text in the HTML spec saying,
“User agents are encouraged to implement the features described in
execCommand”, and also not good to make reference to the execCommand()
API as something that developers should be using directly in their code.
@domenic
Copy link
Member

domenic commented Sep 13, 2021

I disagree with this assessment. The spec cited remains the best documentation/spec we have available for a rather core web platform feature that is extremely widely used. Although nobody has stepped up to maintain that core feature, in an ideal world someone should.

From another perspective: no browser, to my knowledge, has any intent to deprecate this method. So I don't think it's a good idea for HTML to treat it as such, despite the spec being unmaintained.

@sideshowbarker
Copy link
Contributor Author

From another perspective: no browser, to my knowledge, has any intent to deprecate this method. So I don't think it's a good idea for HTML to treat it as such, despite the spec being unmaintained.

From the fact that implementations have not deprecated it, it does not follow that the HTML spec should be actively recommending it. This PR doesn’t change any parts of the spec that normatively reference the execCommand spec. Instead it just drops some incidental, completely discretionary non-normative references. And the reason it drops them is that they conflict with the prominent warning at the top of the execCommand spec itself.

On the face of it, is seems odd for the HTML spec to be even implicitly recommending proliferation of execCommand when the execCommand spec itself is doing the opposite.

@annevk
Copy link
Member

annevk commented Sep 14, 2021

It's not clear to me that warning has implementer agreement. While execCommand() may be in the same realm as XMLHttpRequest, that doesn't mean it doesn't need to be maintained and supported, as well as taken into account when new features come to the scene.

@sideshowbarker sideshowbarker deleted the sideshowbarker/execCommand-non-normative-references-drop branch September 14, 2021 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants