Investigate whether we can add hooks for the webcompat add-on to modify the position and visibility of the Picture-in-Picture toggle
Categories
(Toolkit :: Video/Audio Controls, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox73 | --- | fixed |
People
(Reporter: mconley, Assigned: mconley)
References
(Blocks 1 open bug, Regressed 1 open bug)
Details
Attachments
(2 files)
There are certain sites where the toggle interferes with elements on screen in ways that users might find jarring or unintuitive.
Our hypothesis is that we can special-case these sites using the webcompat add-ons intervention capability. We need to investigate this, and if it's possible, add the hooks necessary to do this special-casing.
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
The webcompat-addon is not only used in Firefox Desktop, but we also ship the very same code to Fennec and android-components (and the component is already enabled in Fenix), which is why development on that addon does usually not happen in m-c, but on GitHub. It's fine for patches to be submitted against m-c, but we have to keep in mind that the code has to run on more than one platform, unless this is a very short-lived experiment.
This looks fine, but could you please do us the favor of designing the experimental API in a way that just gracefully fails if the APIs in question are not there? aboutConfigPrefs.getPref()
, for example, eats all exceptions and returns undefined
, resulting in the addon progressing as if nothing ever happened. The other functions are async, resulting in a failed promise, but nothing more serious. pictureInPicture.getAPI()
looks like it could be unhappy inside android-components the way it's currently build.
Thank you :)
Updated•6 years ago
|
Assignee | ||
Comment 3•6 years ago
|
||
Depends on D57177
Comment 6•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/38ad56dbac9b
https://hg.mozilla.org/mozilla-central/rev/a3701e47b259
Description
•