Bug 1639218 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Since we don't have the need to switch all prompts back to window prompts we should remove the `prompts.tab_modal.enabled`. Right now every test that shows a tab prompt has to check for this pref.

If we want to control the modal type, we can add individual prefs like we already have for some prompts:
`prompts.modalType.<promptName>`
Revision 1 by
on 
Since we don't have the need to switch all prompts back to window prompts we should remove the `prompts.tab_modal.enabled`. Right now every test that shows a tab prompt has to check for this pref. Also having a boolean pref here doesn't make sense anymore, since there are now 3 modal types for prompts: window, tab and content.

If we want to control the modal type, we can add individual prefs like we already have for some prompts:
`prompts.modalType.<promptName>`

Back to Bug 1639218 Comment 0