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

HTML: <object usemap> is removed #27184

Open
wants to merge 1 commit into
base: master
from
Open

HTML: <object usemap> is removed #27184

wants to merge 1 commit into from

Conversation

@annevk
Copy link
Member

@annevk annevk commented Jan 14, 2021

Tests for whatwg/html#6283.

So the curious case here is html/semantics/forms/the-label-element/clicking-interactive-content.html where browsers fail for <object> but not <object usemap>. If I add <span> or <div> I get the same results (only the last of the three tests fails) so I suspect that all browsers do have some logic to consider <object usemap> interactive? Despite Chrome and Safari not supporting <object usemap> otherwise.

Tests for whatwg/html#6283.
@wpt-pr-bot wpt-pr-bot added the html label Jan 14, 2021
@github-actions github-actions bot requested a deployment to wpt-preview-27184 Jan 14, 2021 Pending
@annevk
Copy link
Member Author

@annevk annevk commented Jan 14, 2021

(You could imagine more cases by the way such as what happens when the object element does have a nested browsing context and whether that is considered interactive. Generally coverage seems pretty poor.)

@annevk annevk mentioned this pull request Jan 14, 2021
1 of 3 tasks complete
@domenic
Copy link
Member

@domenic domenic commented Jan 21, 2021

Ping @mfreed7 on whether you're interested in this minor <object> cleanup.

@mfreed7
Copy link
Contributor

@mfreed7 mfreed7 commented Feb 26, 2021

Yes, I'm definitely interested in this cleanup. We have no use counters, but this seems rather safe.

In Chromium, this is made interactive here, but I don't see any other behavioral changes to actually do anything other than that. Happy to remove that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants