-
Notifications
You must be signed in to change notification settings - Fork 0
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
Reusing allow="" is a bit weird #5
Comments
I'm trying to reuse as much as possible that already exists, this was meant to be analogous in terms of giving the link permission to perform some action "open multiple windows", but yeah, the tokens wouldn't be the same so point taken. Taking another look at this...I see we can specify the content type for an anchor using the |
Wow, fascinating, I'd totally forgotten In general, I think it's OK to invent new things for new features, instead of trying to find clever ways of reusing or repurposing existing things. |
Having thought about it more, I take back A new token for the Another option is to add a boolean attribute: +@vmpstr for thoughts as well. Another thing to keep in mind is we'll likely also need this on |
allow="" already has a meaning related to permissions policy and
<iframe>
. Reusing it for<a>
, presumably with a different set of tokens (not related to permissions policy), seems weird.The text was updated successfully, but these errors were encountered: