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

Document.hasFocus, ChildNode.remove are not supported by Opera 12.18 #3165

Merged
merged 2 commits into from Dec 3, 2018

Conversation

Projects
None yet
2 participants
@Abradoks
Copy link
Contributor

Abradoks commented Dec 3, 2018

I've tested these 2 methods in Opera 12.18 (Presto) and they are not supported. I did not test the support in Opera 15 (Blink).

This might apply to Opera Mobile (opera_android) as well, but I didn't test it.

Note that Presto docs claim compatibility. #2678 was based on those docs. There might be more incorrect compat data based on those docs.

I also have a general concern about Opera Presto and Blink sharing the same compat id. While Presto definitely deserves its place in the table (being one of the very few browser engines), Opera Blink simply reflects the Chrome column, like myriads of other Chrome clones.

Abradoks added some commits Dec 3, 2018

@ddbeck

ddbeck approved these changes Dec 3, 2018

@ddbeck

This comment has been minimized.

Copy link
Collaborator

ddbeck commented Dec 3, 2018

Hi @Abradoks, welcome to BCD. This change looks fine to me, so I'm going to merge it. Thank you!

Also regarding this:

I also have a general concern about Opera Presto and Blink sharing the same compat id. While Presto definitely deserves its place in the table (being one of the very few browser engines), Opera Blink simply reflects the Chrome column, like myriads of other Chrome clones.

My recollection of past discussions on this topic is that some user research showed that web developers generally want to know whether a browser version supports a feature, not whether a rendering engine does, which is why we group data the way we do. In other words, Presto may have ceased development, but Opera didn't. Plus, the project gets a steady stream of contributions on both Chrome and Opera, making supporting that data choice sustainable.

@ddbeck ddbeck merged commit 90e75a0 into mdn:master Dec 3, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment