Warn user if PiP video is playing when quitting browser |
|||||||||||
Issue descriptionIt would be great to warn user when they're about to quit browser if a PiP video is playing. See what Safari is doing.
,
Feb 7 2018
mlamourni@ yes, that was the intention. I don't feel strongly about the dialog, but I'd note that we don't currently pop one when closing a page playing audio or when in a hangout.
,
Feb 8 2018
I believe it is especially useful as the PiP video doesn't have any Chrome UI. User may think it is not related to Chrome and simply want to continue playing video while closing browser that is already in a minimized state.
,
Feb 8 2018
Fair point. +srahim for guidance on what the dialog should say.
,
Feb 10 2018
Quick suggestion for something less verbose than the Safari dialog: Quit Chrome? The picture-in-picture video will stop playing. Cancel | Quit
,
Feb 10 2018
,
Feb 12 2018
fbeaufort@ does Safari show a different message when closing a window (ie. multiple tabs)? or just closing the tab playing in PIP? In other words, I'm wondering when we should show the message? Only when closing Chrome or when closing the tab regardless of the context?
,
Feb 12 2018
Messages are indeed different when closing a single tab and a window. See screenshots attached.
,
Feb 12 2018
srahim@ that sounds good to me when closing a window. Should it be worded any differently when closing a tab?
,
Feb 26 2018
,
Mar 27 2018
Hm, I'd say the message could be the same regardless of quitting a tab or a window - the main point is you're about to quit Chrome and the PIP will stop playing. Unless I'm missing something, I think it's OK to use the same message for both tab or window.
,
Mar 27 2018
,
Apr 11 2018
Proposal SGTM. Assigning to Jennifer to implement.
,
Apr 11 2018
Should we also show the warning when navigating away from the page we triggered PiP? See screenshot for slightly different messaging.
,
May 31 2018
,
Jun 22 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e45a7339b7867f398e271775bc2c495fee3b5b3b commit e45a7339b7867f398e271775bc2c495fee3b5b3b Author: Jennifer Apacible <apacible@chromium.org> Date: Fri Jun 22 20:42:35 2018 [Picture in Picture] Add strings for confirmation dialog. These strings will be used for the dialog that appears when the user: - is closing the tab or window that initiated a Picture-in-Picture session, or - is navigating away from a page that initiated a Picture-in-Picture session. Dialog implementation is in progress. Bug: 809982 TBR: grt@chromium.org Change-Id: I060fb2f21fc93361aeb11f4b605b5012d369e28c Reviewed-on: https://chromium-review.googlesource.com/1112304 Commit-Queue: apacible <apacible@chromium.org> Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#569750} [modify] https://crrev.com/e45a7339b7867f398e271775bc2c495fee3b5b3b/chrome/app/generated_resources.grd
,
Jul 5
,
Jul 10
,
Jul 12
The team is moving towards deprecating warning dialogs. In a similar vein as not showing any warnings when the Casting tab is closed, we will not show warnings for now. Open to revisiting if we get overwhelming feedback from users that this is a problem. |
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by mlamouri@chromium.org
, Feb 7 2018