Closed
Bug 801891
Opened 12 years ago
Closed 12 years ago
Fullscreen approval UI should be modal
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 19
People
(Reporter: cpearce, Assigned: cpearce)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
The fullscreen approval UI dialog should be modal, i.e. the user should not be able to interact with the page while the <browser> has dimmed its content while the fullscreen approval dialog is showing. This is to force the user to pay attention to the dialog, and increase the likelihood the user actually reads it.
I'd previously been using pointer-events:none thinking that it would achieve this, but it turns out I didn't test it properly and pointer-events:none isn't inherited into descendents of the <browser>, so I think we should use -moz-user-focus:none, which is what we use to make window.alert() modal.
Assignee | ||
Comment 1•12 years ago
|
||
Use -moz-user-focus:none; instead of pointer-events:none, so that users can't interact with web pages while the fullscreen approval UI is showing. This forces them to acknowledge the approval UI, making it more likely that they actually read it.
Attachment #671621 -
Flags: review?(dao)
Comment 2•12 years ago
|
||
Comment on attachment 671621 [details] [diff] [review]
Patch: Make fullscreen approval UI modal
> #full-screen-warning-container {
>- pointer-events: none;
>+ -moz-user-focus: none !important;
> position: fixed;
> top: 0;
> left: 0;
> width: 100%;
> height: 100%;
> }
This node should already be unfocusable by default, -moz-user-focus:none should be a no-op. Seems like you just want to drop pointer-events:none.
(In reply to Chris Pearce (:cpearce) from comment #0)
> I'd previously been using pointer-events:none thinking that it would achieve
> this, but it turns out I didn't test it properly and pointer-events:none
> isn't inherited into descendents of the <browser>, so I think we should use
> -moz-user-focus:none, which is what we use to make window.alert() modal.
You weren't setting pointer-events:none on the browser, so it can't have any effect on that. As opposed to the tab-modal alert() UI, you're not setting -moz-user-focus:none on the browser either.
Attachment #671621 -
Flags: review?(dao) → review-
Assignee | ||
Comment 3•12 years ago
|
||
Thanks for your comments Dão, removing pointer-events:none does the trick.
Attachment #671621 -
Attachment is obsolete: true
Attachment #672020 -
Flags: review?(dao)
Comment 4•12 years ago
|
||
Comment on attachment 672020 [details] [diff] [review]
Patch v2: Remove pointer-events:none from fullscreen approval UI
This is probably good enough for now, although I suppose the content area can still be reached with the TAB key.
Attachment #672020 -
Flags: review?(dao) → review+
Assignee | ||
Comment 5•12 years ago
|
||
Comment 6•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 19
You need to log in
before you can comment on or make changes to this bug.
Description
•