Closed Bug 801891 Opened 12 years ago Closed 12 years ago

Fullscreen approval UI should be modal

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 19

People

(Reporter: cpearce, Assigned: cpearce)

References

Details

Attachments

(1 file, 1 obsolete file)

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.
Attached patch Patch: Make fullscreen approval UI modal (obsolete) (deleted) — Splinter Review
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 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-
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 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+
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.

Attachment

General

Created:
Updated:
Size: