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

Allow <a>/<area> with download="" to not require user activation #2136

Merged
merged 1 commit into from
Dec 6, 2016

Conversation

domenic
Copy link
Member

@domenic domenic commented Dec 5, 2016

Fixes #2116, wherein it is documented that this restriction is not
supported in any existing engines and attempting to do so in WebKit
proved not web-compatible.

Also helps with #1431.

Tests: web-platform-tests/wpt#4283

Fixes #2116, wherein it is documented that this restriction is not
supported in any existing engines and attempting to do so in WebKit
proved not web-compatible.

Also helps with #1431.
@domenic domenic added the needs tests Moving the issue forward requires someone to write tests label Dec 5, 2016
@domenic domenic mentioned this pull request Dec 5, 2016
31 tasks
domenic added a commit to web-platform-tests/wpt that referenced this pull request Dec 5, 2016
That is, that calling .click() does not throw at least, as it used to
per-spec. Matches the HTML change at
whatwg/html#2136 per the HTML bug report at
whatwg/html#2116.
@domenic domenic removed the needs tests Moving the issue forward requires someone to write tests label Dec 5, 2016
@zcorpan
Copy link
Member

zcorpan commented Dec 6, 2016

LGTM

@domenic domenic merged commit 5de03c7 into master Dec 6, 2016
@domenic domenic deleted the a-download branch December 6, 2016 23:33
data-x="attr-hyperlink-target">target</code> attribute is present, and applying <span>the rules
for choosing a browsing context given a browsing context name</span>, using the value of the
<code data-x="attr-hyperlink-target">target</code> attribute as the browsing context name, would
result in there not being a chosen browsing context, then run these substeps:</p>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This restriction was not just about the download attribute. And the commit message doesn't acknowledge the other bit? Did you mistakenly remove too much here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, sloppy reviewing on my part. I guess I glossed over "or" and read it as "and". ☹️ We just broke the popup blocker and sandbox...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

domenic added a commit to web-platform-tests/wpt that referenced this pull request Feb 8, 2017
That is, that calling .click() does not throw at least, as it used to
per-spec. Matches the HTML change at
whatwg/html#2136 per the HTML bug report at
whatwg/html#2116.
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.

3 participants