507122 - toolbar style don't match Firefox

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect)

Description

[Andreas Nilsson (:andreasn)]

The style of the toolbar, with a line between the menu and toolbar, don't match the one on Firefox that sports a more united appearance.

Comment 1

[Andreas Nilsson (:andreasn)]

Attachment: patch on trunk

Here is a patch that fixes this issue.

Comment 2

[Andreas Nilsson (:andreasn)]

Attachment: screenshot of the patch in action

Comment 3

[Bryan Clark (DevTools PM) [:clarkbw]]

Attachment: andreas patch uses %include instead

Has this been testing in XP?  maybe I can get philor to look at this in XP

I like the new toolbar look but I'm wondering if we need a messenger-aero.css file when the difference of the two is so small.

Here's what I have for the diff of the two files:

--- messenger.css       Wed Aug  5 09:54:45 2009
+++ messenger-aero.css  Wed Aug  5 09:54:45 2009
@@ -13,7 +13,9 @@

 .toolbar-primary {
   -moz-binding: url("chrome://global/content/bindings/toolbar.xml#toolbar");
+  -moz-appearance: none;
   border-top-style: none;
+  border-top-width: 0px;
 }

I've included an alternate style patch to this problem.  Essentially when most of the windows XP and Vista CSS is going to be shared we should be using this style that I learned from philor.

The differences are this:
jar.mn has a "*" in front of the line which tells the compile to pre-process the file.  Essentially meaning it will use the include to make the new file with all the CSS rules appended to the bottom of the new file.  This works really well for CSS.

messenger.css only has the different CSS rules that are needed for the Vista components.

What does suck about this is that from now on any line numbers related to CSS errors in Vista can be wrong.  However the aero override file should be small and if the line number is larger than the file you know you're looking at the aero file instead of the original included file.

Comment 4

[Bryan Clark (DevTools PM) [:clarkbw]]

maybe I should add philor if I want him to look at the patch

Comment 5

[Bryan Clark (DevTools PM) [:clarkbw]]

Attachment: fix spelling messs

also you should try this one since my misspelling of messenger makes that other patch not work

Comment 6

[Bryan Clark (DevTools PM) [:clarkbw]]

Comment on attachment 391330 [details] [diff] [review]
patch on trunk

minus for now until we get the code sorted out

Comment 7

[Bryan Clark (DevTools PM) [:clarkbw]]

Comment on attachment 392758 [details] [diff] [review]
fix spelling messs 

hmmm, `something clever` and can you take a look at this?

Comment 8

[Phil Ringnalda (:philor)]

Comment on attachment 392758 [details] [diff] [review]
fix spelling messs 

Looks good to me.

Comment 9

[Phil Ringnalda (:philor)]

http://hg.mozilla.org/comm-central/rev/1a2824475537