Closed
Bug 1812939
Opened 2 years ago
Closed 2 years ago
PiP controls are not removed after adjusting subtitles font setting
Categories
(Toolkit :: Picture-in-Picture, defect)
Tracking
()
VERIFIED
FIXED
112 Branch
People
(Reporter: asafko, Assigned: niklas)
References
Details
(Whiteboard: [fidefe-pip3])
Attachments
(1 file)
Reproducible on macOS Monterey 12.4 and 12.5
STR
- Launch a video with captions/subtitles on in a PiP.
- Click the subtitles button in the PiP window, and change the
font size. Click on the video to close the subtitles settings panel. - Stop hovering over the PiP window.
Expected result
PiP playback controls disappear once the user starts hovering over the PiP window.
Actual result
PiP playback controls remain visible until the PiP window is closed or unpopped.
Updated•2 years ago
|
| Assignee | ||
Comment 1•2 years ago
|
||
Updated•2 years ago
|
Assignee: nobody → nbaumgardner
Status: NEW → ASSIGNED
Pushed by nbaumgardner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c9d22ed07b44
Remove donthide attribute from controls when hiding subtitles settings. r=pip-reviewers,mhowell
Comment 3•2 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox112:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
Comment 5•2 years ago
|
||
This behavior appears to be fixed in Nightly v112.0a1 in Windows 10, MacOS 11 and Ubuntu 22.
Should this fix be considered for uplift?
Flags: needinfo?(nbaumgardner)
Updated•2 years ago
|
status-firefox110:
--- → affected
status-firefox111:
--- → affected
status-firefox-esr102:
--- → affected
Updated•2 years ago
|
OS: macOS → All
| Assignee | ||
Comment 6•2 years ago
|
||
I don't think this requires an uplift
Flags: needinfo?(nbaumgardner)
Comment 7•2 years ago
|
||
The patch landed in nightly and beta is affected.
:niklas, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox111towontfix.
For more information, please visit auto_nag documentation.
Flags: needinfo?(nbaumgardner)
| Assignee | ||
Updated•2 years ago
|
Flags: needinfo?(nbaumgardner)
Updated•2 years ago
|
Updated•2 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•