Don't turn off shadows for transparent windows

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
10 years ago
10 years ago

People

(Reporter: mstange, Assigned: mstange)

Tracking

Trunk
All
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 obsolete attachment)

(Assignee)

Description

10 years ago
Created attachment 333213 [details] [diff] [review]
fix

In bug 307204 it was decided to turn off shadows for transparent windows for the following reasons:
 1) There might be problems with updating the shadows for changing content.
 2) Invalidating shadows on every drawRect is expensive.

However, I need shadows for transparent windows for rounded context menus (bug 391984) and I'd like to have them without having to tackle bug 415443.

I haven't seen any problems with this patch in my tests on both Tiger and Leopard. And I think performance isn't that important in this case - normal windows aren't transparent, and those that are / will be (bookmarks and larry panel, context menus) aren't performance critical in any way; I also haven't felt any difference in snappiness.
Attachment #333213 - Flags: superreview?(roc)
Attachment #333213 - Flags: review?(hwaara)
(Assignee)

Comment 1

10 years ago
With the patch all windows / panels have shadows now (the larry and bookmarks popup didn't have them before), which I think is nice. I'm not aware of any Firefox windows that shouldn't have shadows.

Comment 2

10 years ago
So we're changing the default here. 

I'd like to know if the current users of transparent windows will be surprised/angry if we suddenly turn them on, without any ability to turn them off (basically bug 415443)...
(Assignee)

Comment 3

10 years ago
I'd like to know that, too - I don't know any users of transparent windows.
But they might well be... I guess it's better to just fix bug 415443.
So we don't want this patch?
(Assignee)

Comment 5

10 years ago
Comment on attachment 333213 [details] [diff] [review]
fix

Yeah, looks like it.
Attachment #333213 - Attachment is obsolete: true
Attachment #333213 - Flags: superreview?(roc)
Attachment #333213 - Flags: review?(hwaara)
(Assignee)

Updated

10 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → WONTFIX

Comment 6

10 years ago
(In reply to comment #3)
> I'd like to know that, too - I don't know any users of transparent windows.
> But they might well be... I guess it's better to just fix bug 415443.

Both the new security UI "Larry" and Places use transparent windows in Firefox 3. I know Songbird uses lots of transparent windows, and I suspect many other XULRunner users as well as extensions, that's why I wouldn't want to flip the default switch without the option to turn the shadows off.
You need to log in before you can comment on or make changes to this bug.