Closed Bug 546259 Opened 15 years ago Closed 15 years ago

enable aero glass for the main window

Categories

(Firefox :: Theme, enhancement)

x86
Windows 7
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 3.7a4

People

(Reporter: dao, Assigned: dao)

References

(Depends on 1 open bug)

Details

Attachments

(2 files, 2 obsolete files)

No description provided.
Attached patch patch (obsolete) (deleted) — Splinter Review
Assignee: nobody → dao
Status: NEW → ASSIGNED
Attachment #434265 - Flags: review?(rflint)
Attached patch patch (deleted) — Splinter Review
updated to tip
Attachment #434265 - Attachment is obsolete: true
Attachment #434493 - Flags: review?(rflint)
Attachment #434265 - Flags: review?(rflint)
@Dao: I tried using this code via Stylish to apply Glass... it worked well, except for one problem... - Apply Glass via Stylish - Press 'Alt' to show the Menubar momemtarily - Leave 'Alt' so that the toolbars go into their original position - Look at the area just below Navigator-Toolbox... it has remnants from the Glassy era :)... with the same thickness as the Menubar. Now this could be because I used it through stylish... may not be the case, once its used as a part of the browser code... is there an experimental build to try out?
It's not because of Stylish but a bug in the fix for bug 450767. Would you mind filing a new bug?
Comment on attachment 434493 [details] [diff] [review] patch Patch looks good, but we should make sure that bug 458407 is going to get attention before landing this. Dynamically changing the window appearance (f.e. applying a persona) also causes some strangeness, I've filed bug 554764 for that.
Attachment #434493 - Flags: review?(rflint) → review+
Depends on: 554770
(In reply to comment #5) > It's not because of Stylish but a bug in the fix for bug 450767. Would you mind > filing a new bug? Filed #554770
(In reply to comment #6) > (From update of attachment 434493 [details] [diff] [review]) > Patch looks good, but we should make sure that bug 458407 is going to get > attention before landing this. I don't think that this bug should affect the theme (though it does it seems because the patch for bug 450767 seems to have a bug where it doesn't determine the opaque regions correctly).
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a4
Sorry to comment in a fixed bug , i've just tried the nightly with this check and i get this, i guess a screenshot is better than words http://img121.imageshack.us/img121/1306/uglye.jpg After browsing abit it gets worse, like the glass area isn't repainted anymore http://img109.imageshack.us/img109/344/ugly2.jpg
(In reply to comment #10) > Sorry to comment in a fixed bug , i've just tried the nightly with this check > and i get this, i guess a screenshot is better than words > > http://img121.imageshack.us/img121/1306/uglye.jpg > > After browsing abit it gets worse, like the glass area isn't repainted anymore > > http://img109.imageshack.us/img109/344/ugly2.jpg Can you please file a new bug?
Sure i will , even tho my english isn't very good, I just noticed that this only happen with d2d/dw on , disabling them everything looks fine. So it's an iteration with d2d/dw
According to the mockups (https://wiki.mozilla.org/images/3/35/Firefox-4-Mockup-i05-%28Win7%29-%28Aero%29-%28BottomTabs%29.png), there wont be Aero borders (the white line around page content), while this implementation does have them. Is there a separate bug for this?
no...
After hiding bars , browser implements aero dwm on content too! So aero is there , but functionality is uncertain. Maybe this is wrong place to inform , still
Image reference of DWM and content problem
(In reply to comment #15) > After hiding bars , browser implements aero dwm on content too! So aero is > there , but functionality is uncertain. Maybe this is wrong place to inform , > still There's bug 554770 filed on that already.
sorry if i caused inconvenience
Depends on: 554944
Depends on: 554975
Depends on: 554982
Depends on: 555063
Depends on: 555081
Attached image The new aero effect causes this (obsolete) (deleted) —
I am seeing this with the latest nightly. I can't imagine that it is supposed to look like this, it's hard for the eyes and isn't close to what the mock-ups show.
Are you using a black background ?
That's the appearance of all maximized windows on Vista. IE8 has a black toolbar as well, but a brighter (non-glass) tab bar.
IE8 has a black *navigation* toolbar as well.
Oh, right. Well, Minefield certainly wasn't ready for it. But I guess it'll get better when more progress is done. I'll use a Persona for now, as I can't stand the look.
Depends on: 555182
Note that only Windows Vista makes the glass parts look black on maximized windows. On Windows 7, there's no difference between maximized and not-maximized windows.
(In reply to comment #24) > Note that only Windows Vista makes the glass parts look black on maximized > windows. On Windows 7, there's no difference between maximized and > not-maximized windows. I see. Do we have the control required to make Vista behave the same way?
Blocks: 555427
No longer blocks: 555427
Well, the discussion about color of screen background and Vista sometimes reverting to displaying black is entirely irrelevant here. The real issue is that this is semi-transparent and will show whatever color the window behind Firefox might be displaying at this position, not necessarily the desktop background. Therefore, unfortunately, toolbar text really needs to be readable regardless of the color displayed behind the window.
only way to do that is to have toolbar text change to light when backgound black and vice versa or disable semi-transparen
(In reply to comment #26) > Well, the discussion about color of screen background and Vista sometimes > reverting to displaying black is entirely irrelevant here. How is it irrelevant? If Firefox.next got released looking like that, people would almost be unable to use it.
(In reply to comment #26) > The real issue is that this is semi-transparent and will show whatever color > the window behind Firefox might be displaying at this position, not necessarily > the desktop background. Therefore, unfortunately, toolbar text really needs to > be readable regardless of the color displayed behind the window. Yes and that's why this is clearly against the Windows User Experience Interaction Guidelines: "Consider using glass strategically in small regions touching the window frame without text. [...] Don't use glass in situations where a plain window background would be more attractive or easier to use." (http://msdn.microsoft.com/en-us/library/aa974173.aspx#glass)
(In reply to comment #29) > (In reply to comment #26) > > The real issue is that this is semi-transparent and will show whatever color > > the window behind Firefox might be displaying at this position, not necessarily > > the desktop background. Therefore, unfortunately, toolbar text really needs to > > be readable regardless of the color displayed behind the window. > > Yes and that's why this is clearly against the Windows User Experience > Interaction Guidelines: > > "Consider using glass strategically in small regions touching the window frame > without text. [...] Don't use glass in situations where a plain window > background would be more attractive or easier to use." > > (http://msdn.microsoft.com/en-us/library/aa974173.aspx#glass) Microsoft doesn't follow its own guidelines. I wouldn't expect anyone else to do so either. The best case would be for Mozilla to come up with a solution of its own. Using the blueish background that's in the mockups and activate it for the whole window when the window is maximized would be a worthy compromise for Vista users.
(In reply to comment #30) > Microsoft doesn't follow its own guidelines. Mostly they do, especially in this case here. Just have a look how the glass is used in Windows and you'll see that it's not used in that amount and not for regions with text on it. >I wouldn't expect anyone else to > do so either. The use of glass as described in the guidelines are not described just because someone wants it to be used that way but because there are good reasons not to use glass on regions with text and in an extensive amount. Even if the guidelines wouldn't say anything about that it would still be a bad decision. > The best case would be for Mozilla to come up with a solution of > its own. Using the blueish background that's in the mockups That would be a reasonable solution but another solution would be to use the glass on the navigation toolbar only, imho. That's also the way IE and others handle it. I have never seen a program with that much glass.
The mockups looks great in my opinion. The issue here is that the current builds look nothing like them. But that will change and get better as progress is made towards the official release.
(In reply to comment #31) > Mostly they do, especially in this case here. Just have a look how the glass is > used in Windows and you'll see that it's not used in that amount and not for > regions with text on it. So, you should have a look at Office 2010. Currently, the text on aero glass (menu bar, caption etc.) is perfectly readable whatever the background (dark or light), due to the glow arround the text.
In Office 2010 the glass is made brighter and not so transparent so that the text is readable. I think it looks good in the mockups and think that most of us here will agree that the current implementation is not the last word on the subject. I just wanted to say that using the glass on the navigation bar only would be another solution for this. :-)
(In reply to comment #31) > (In reply to comment #30) > Mostly they do, especially in this case here. Just have a look how the glass is > used in Windows and you'll see that it's not used in that amount and not for > regions with text on it. The Gadgets Window still looks the same in Windows 7 despite the guidelines. Otherwise most of its default apps interfaces have never been updated. Those that they do update are going to the Ribbon UI. Of which ironically puts controls on the glass window frame, violating the guidelines. > The use of glass as described in the guidelines are not described just because > someone wants it to be used that way but because there are good reasons not to > use glass on regions with text and in an extensive amount. Even if the > guidelines wouldn't say anything about that it would still be a bad decision. Microsoft uses itself in the demonstrations on how not to use glass. In the guideline, they are saying that it shouldn't be used for everything. I understand that. But in the case of the final Fx4.0 mockup, its not abused IMO. In a couple of those cases they mentioned, they clearly used glass despite those guidelines. Even the Windows 7 taskbar and Start Menu violates those directives. > That would be a reasonable solution but another solution would be to use the > glass on the navigation toolbar only, imho. That's also the way IE and others > handle it. I have never seen a program with that much glass. Try turning on Opera 10.51's toolbars. SiSoftware Sandra uses glass extensively. Outside of that, I haven't seen many apps that were bothered to update themselves to use the new Windows UI stuff period. You would have to pry Windows Xp from their cold, dead fingers first.
IMHO either our glow needs to be stronger (on windows 7 anyways - compare the title bar glow with Firefox's glow) or we should drop the glow entirely and make the text white (see windows 7 start menu for precedent - even when you set your glass to white there's still a bit of a 'black glow' behind there)
(In reply to comment #36) > IMHO either our glow needs to be stronger (on windows 7 anyways - compare the > title bar glow with Firefox's glow) or we should drop the glow entirely and > make the text white (see windows 7 start menu for precedent - even when you set > your glass to white there's still a bit of a 'black glow' behind there) Ah if whatever is being displayed behind that portion of the Firefox window is solid white, exactly how is making the text white going to help? From what I see, today;s changes have improved the situation dramatically. The issue I see still remaining, is that it is still hard to read the Main window menu in the case where the menu toolbar is not hidden and the Firefox window does not have focus. Perhaps dimming the main menu items when the window is not focused should only be done in the non-glass case?
How is it fixed if there are issues related to aero? check my comments @ Bug 554770
Depends on: 555845
Attachment #435126 - Attachment is obsolete: true
dao - didn't you back this out?
(In reply to comment #40) > No, but I pushed http://hg.mozilla.org/mozilla-central/rev/69e17a1b3034 Alright, so this was fixed, but it has been disabled until bug 555182 is resolved.
Depends on: 556967
blocking2.0: ? → ---
Depends on: 560195
bug 555182 is now fixed
Depends on: 567742
Depends on: 567771
Depends on: 569156
No longer depends on: 569156
Blocks: 575942
No longer blocks: 575942
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: