Closed Bug 1466296 Opened 6 years ago Closed 6 years ago

Combine videoControls and noControls binding

Categories

(Toolkit :: UI Widgets, task, P5)

Product:
Toolkit
Component:
UI Widgets
Type:
task

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1431255

People

(Reporter: timdream, Unassigned)

References

Details

Bug 1431255 will remove the ability for the JS widget to bind/unbind based on CSS selector matching, i.e. what XBL offers.

I think we should simply it first by always bind the binding under <videocontrols>, and have the JS to decide if it wants to show itself.

Doing so involves merging videoControls and noControls binding, and detect changes to the "controls" attribute (either by listening to an event, or mutation observer).

This itself is complex enough so I wish I can land it separately, hence another bug.
Not my first priority, feel free to grab.
Assignee: timdream → nobody
Status: ASSIGNED → NEW
I've figured out some other way to do this in bug 1431255.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
Resolution: INVALID → DUPLICATE
Type: enhancement → task
You need to log in before you can comment on or make changes to this bug.