Closed Bug 2006594 Opened 7 months ago Closed 3 months ago

Enable Document Picture-in-Picture on Desktop in release

Categories

(Core :: DOM: Core & HTML, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
151 Branch
a11y-review changes required
Tracking Status
relnote-firefox --- 151+
firefox151 --- fixed

People

(Reporter: vhilla, Assigned: vhilla)

References

(Blocks 3 open bugs)

Details

(Keywords: dev-doc-complete, webcompat:platform-bug)

Attachments

(2 files)

No description provided.
Depends on: 1858562, 2006597
Depends on: 2006642
Depends on: 2008334
Depends on: 2006849
See Also: → 2009192
Depends on: 2009660
Keywords: dev-doc-needed
See Also: → 2012726
Blocks: doc-pip-api
No longer depends on: 2006597
See Also: → 2006597
Depends on: 2011409
See Also: → 2016431
Assignee: nobody → vhilla
Status: NEW → ASSIGNED

Description:

The Document Picture-in-Picture API is an always-on-top popup. It has an additional "Return" button to get back to the opener and some browser UI elements are hidden.
It's an API exposed by the web, therefore user scenarios depend on the page. Pages might put audio controls in this popup, show a custom media player rather than using the existing Video Picture-in-Picture, or do simple things like showing a stopwatch. In general: The user interacts with the page, the popup opens and gains focus, the user interacts with the popup and at some point (a) closes it like any other window (b) closes it via the return button so focus moves back to the opener page or (c) an action is performed that auto-closes the popup.

How do we test this?

The API is behind the dom.documentpip.enabled pref that is enabled nightly-only. I'll add a simple test page to this bug.

When will this ship?

Soon, if bug 2008334 lands and the a11y review completes.
Tracking bug/issue: this bug

Please describe the accessibility guidelines you considered and what steps you've taken to address them:

I went through the guidelines to make sure we follow them. Controls are semantically hidden, buttons come from the native host language. The image icon on the return button doesn't have an alt text, but other browser UI buttons don't have either.

Describe any areas of concern to which you want the accessibility team to give special attention:

  • Always-on-top windows aren't new to Firefox, but maybe there are concerns about them?
  • The spec requires the window to be closed on certain disallowed operations. E.g. if the user navigates or reloads the opener or popup. Could the window disappearing be an issue? Also see bug 2019895.
  • There is a new "Return" button introduced in the browser UI.
a11y-review: --- → requested
Attached file example.html
a11y-review: requested → assigned

Hey Vincent! Thank you for sending this over (: I'm starting on this now but if you need anything, please let me know!

Depends on: 2023195
Depends on: 2023637
Depends on: 2023649
Depends on: 2023668
Depends on: 2023678
a11y-review: assigned → changes required
Depends on: 2024297
No longer depends on: 2024297
No longer depends on: 2023649
No longer depends on: 2023637
No longer depends on: 2023668

Regarding dev-doc-needed, see bug 1858562#c26. I think it only needs to be added to the release notes.


For the release note request, I'm paraphrasing 1858562#c22:

[Why is this notable]: New web API
[Affects Firefox for Android]: No
[Suggested wording]: Firefox now supports the Document Picture-in-Picture API, which allows web pages to place content in an always-on-top popup.
[Links (documentation, blog post, etc)]: https://developer.mozilla.org/en-US/docs/Web/API/Document_Picture-in-Picture_API

relnote-firefox: --- → ?
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 151 Branch
Summary: Enable Document Picture-in-Picture in release → Enable Document Picture-in-Picture on Desktop in release

Added to the Fx151 nightly release notes, please allow 30 minutes for the site to update.
Keeping the relnote-firefox flag as ? to keep it on the radar for inclusion in the final Fx151 release notes.

Regressions: 2032464

FF151 MDN work for this can be tracked in https://github.com/mdn/content/issues/43861. Docs are good. This is just a release note an compatibility data update.

QA Whiteboard: [qa-triage-done-c152/b151]

Added to the final Fx151 release notes

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: