Awesome, thanks for the quick turnaround time! With that change, LGTM1. On Fri, Mar 15, 2024 at 2:16 AM Tommy Steimel <stei...@google.com> wrote:
> Thanks for the feedback! Updated in > https://github.com/WICG/document-picture-in-picture/pull/116 > > On Wed, Mar 13, 2024 at 6:53 PM Domenic Denicola <dome...@chromium.org> > wrote: > >> I found an issue with the API design here that might result in a >> backward-incompatible change: >> https://github.com/WICG/document-picture-in-picture/issues/115 >> >> With my spec mentor hat on, sorry for not catching it sooner! >> >> On Thu, Mar 14, 2024 at 2:12 AM 'Tommy Steimel' via blink-dev < >> blink-dev@chromium.org> wrote: >> >>> Contact emailsstei...@chromium.org, liber...@chromium.org >>> >>> ExplainerNone >>> >>> Specification >>> https://github.com/WICG/document-picture-in-picture/pull/114 >>> >>> Summary >>> >>> This adds a new parameter ("allowReturnToOpener") to the document >>> picture-in-picture API that, when set to false, hints to the user agent >>> that they should not show a button in the picture-in-picture window that >>> allows the user to return to the opener. While having a button to return >>> content to the opener always makes sense in the video picture-in-picture >>> case (the video stream can be returned to the video element in the opener >>> tab), this is not always the case for document picture-in-picture >>> experiences. This gives developers more control over the user experience >>> when they determine that such a button does not make sense for their use >>> case. >>> >>> >>> Blink componentBlink>Media>PictureInPicture >>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EMedia%3EPictureInPicture> >>> >>> TAG review >>> https://github.com/w3ctag/design-reviews/issues/798#issuecomment-1967916721 >>> >>> TAG review statusPending >>> >>> Risks >>> >>> >>> Interoperability and Compatibility >>> >>> None >>> >>> >>> *Gecko*: No signal ( >>> https://github.com/mozilla/standards-positions/issues/670#issuecomment-1967919675) >>> Added comment to existing standards position issue for document >>> picture-in-picture. No response yet >>> >>> *WebKit*: No signal ( >>> https://github.com/WebKit/standards-positions/issues/41#issuecomment-1967918830) >>> Added comment to existing standards position issue for document >>> picture-in-picture. No response yet >>> >>> *Web developers*: Positive ( >>> https://github.com/WICG/document-picture-in-picture/issues/113) We have >>> received feature requests for the ability to hide the "back to tab" button >>> from the document picture-in-picture window. >>> >>> *Other signals*: >>> >>> Ergonomics >>> >>> N/A >>> >>> >>> Activation >>> >>> N/A >>> >>> >>> Security >>> >>> N/A >>> >>> >>> WebView application risks >>> >>> Does this intent deprecate or change behavior of existing APIs, such >>> that it has potentially high risk for Android WebView-based applications? >>> >>> N/A, as this is not available on Android >>> >>> >>> Debuggability >>> >>> N/A >>> >>> >>> Will this feature be supported on all six Blink platforms (Windows, Mac, >>> Linux, ChromeOS, Android, and Android WebView)?No >>> >>> The document picture-in-picture API is not supported on Android >>> >>> >>> Is this feature fully tested by web-platform-tests >>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> >>> ?No >>> >>> The document picture-in-picture feature itself is fully tested on WPT, >>> but this additional parameter isn't since it's a hint to the user agent and >>> therefore any actual changes happen in the embedder >>> >>> >>> Flag name on chrome://flagsNone >>> >>> Finch feature nameNone >>> >>> Non-finch justification >>> >>> Small change to existing API >>> >>> >>> Requires code in //chrome?False >>> >>> Sample links >>> https://steimelchrome.github.io/document-pip/hide-back-to-tab-button.html >>> >>> Estimated milestones >>> Shipping on desktop 124 >>> >>> Anticipated spec changes >>> >>> Open questions about a feature may be a source of future web compat or >>> interop issues. Please list open issues (e.g. links to known github issues >>> in the project for the feature specification) whose resolution may >>> introduce web compat/interop risk (e.g., changing to naming or structure of >>> the API in a non-backward-compatible way). >>> N/A >>> >>> Link to entry on the Chrome Platform Status >>> https://chromestatus.com/feature/6223347936657408 >>> >>> This intent message was generated by Chrome Platform Status >>> <https://chromestatus.com/>. >>> >>> -- >>> You received this message because you are subscribed to the Google >>> Groups "blink-dev" group. >>> To unsubscribe from this group and stop receiving emails from it, send >>> an email to blink-dev+unsubscr...@chromium.org. >>> To view this discussion on the web visit >>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAE-AwAqR%2BNBOJT4h9YRkdOB9ksbPYgFCfP5JvmTKuCbFA-4-cQ%40mail.gmail.com >>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAE-AwAqR%2BNBOJT4h9YRkdOB9ksbPYgFCfP5JvmTKuCbFA-4-cQ%40mail.gmail.com?utm_medium=email&utm_source=footer> >>> . >>> >> -- You received this message because you are subscribed to the Google Groups "blink-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscr...@chromium.org. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAM0wra-v7hqnNZrXm866Vq4MJxGX0eCxxEz4k3SYxyLSrDQ%3D1g%40mail.gmail.com.