Consider to change allowfullscreen to allow="fullscreen"

NEW
Unassigned

Status

()

defect
P5
normal
8 years ago
3 months ago

People

(Reporter: smaug, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Reporter

Description

8 years ago
... that way more features, like pointerlock, could be added to allow list.

http://lists.w3.org/Archives/Public/public-webapps/2011OctDec/1569.html

allow="fullscreen pointerlock"
Reporter

Updated

8 years ago
Blocks: 633602
Version: unspecified → Trunk
I'm not as enthusiastic now as I was yesterday.
Reporter

Comment 3

8 years ago
I wonder why. We need something for pointerlock, or otherwise we need to limit it to top level
browsing context.
what's wrong with "allowfullscreen", "allowpointerlock" etc?

It's easier to add, remove and test for existence of independent attributes than for tokens in a space-separated list.
Reporter

Comment 5

8 years ago
how is it easier to remove independent attributes?
I would assume 'allow' will be exposed as a TokenList to JS, same way to 'class'
OK, but that requires extra DOM API.

off the top of my head, I know how to remove attributes, but not from DOMTokenList.
Reporter

Comment 7

8 years ago
element.allow.remove("fullscreen");
Sure, but it's something you have to know.

OTOH I don't care. So I just withdraw any opinion.
No longer blocks: 633602
https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.
Priority: -- → P5
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.