Alice0775 White Reporter
	Description • 14 years ago

Build Identifier: 
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b8pre) Gecko/20101124 Firefox/4.0b8pre ID:20101124042634

Alerts should be content-modal, not window-modal when alert open from Sidebar Panel page


Reproducible: Always

Steps to Reproduce:
1. Start Minefield with new profile
2. Bookmark The URL and Check "Load this bookmark in the sidebar"
3. Opeb the Bookmark in the Sidebar Panel
4. Click "Please Click to pop up Alert" to open alert box


Actual Results:
 The alert box is window-modal.

Expected Results:
 The alert box should be content-modal.

	Nikita Popov
	Comment 1 • 14 years ago

Well, which content should that alert() be modal too? 
You shouldn't make it modal to the main content, because that's not related to the sidebar. 
It would make sense to make it modal to the sidebar, but an alert() doesn't fit in there.

Only way I see is to make it modal to both sidebar and main content. I don't think this is perfect, but at least its better than being window-modal.

	Justin Dolske [:Dolske]
	Comment 2 • 14 years ago

Sidebar is effectively deprecated, and it lacks the bits added in bug 596371 and bug 59314 to support tab-modal overlays. Not going to fix this.