779155 - "Prevent this page from creating additional dialogs" checkbox should be in the grey part of the alert()

Status: NEW

(Toolkit :: Content Prompts, defect)

Description

[Jesse Ruderman]

Attachment: testcase

It's good to separate browser UI from the page's UI.

Suggested by walf in bug 692331 comment 7.

Comment 1

[Jesse Ruderman]

Attachment: screenshot of the problem

Comment 2

[Boris Zbarsky [:bzbarsky]]

I think this is all front-end code, not DOM code...

Comment 3

[Justin Dolske [:Dolske]]

The white/grey wasn't really intended to convey chrome-vs-content, indeed the whole thing should be considered untrusted as in-content UI.

We can't easily do this with the existing prompt code, it's just using the existing prompt type(s) that add a checkbox to the stock UI. It would change all such prompts, which might not be desired. OTOH, at least currently we don't give content the ability too add or control the checkbox text...

Comment 4

[Matthew N. [:MattN]]

Bulk move to Toolkit::Notifications and Alerts

Filter on notifications-and-alerts-component.

Comment 5

[Roy Orbitson]

To a user it's not considered untrusted as you made the native UI look more like content UI. The checkbox should still be moved, and if you want users to trust the checkbox/buttons more, change the modal. E.g.: some overlap with the main UI would let one know it's not just part of the DOM.