Closed Bug 677667 Opened 9 years ago Closed 9 years ago

[Mac default] Make use of the new nsNativeTheme toolbarbutton styling in DM and PageInfo

Categories

(SeaMonkey :: Themes, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.8

People

(Reporter: stefanh, Assigned: stefanh)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch v1.0 (obsolete) — Splinter Review
Toolbar buttons can now be drawn wth of CoreUI (bug 672050). This style is suitable for 'buttons' in DM and pageInfo (we fake it a bit here).

I actually thought bug 672050 wasn't going to get fixed for Mozilla8 (see https://bugzilla.mozilla.org/show_bug.cgi?id=674368#c0), but I was wrong.

I think we need to test this on Lion... But I don't have Lion.
Attached patch v1.1Splinter Review
Noted that the "toolbar" in pageInfo was slightly higher than it should be. I also now use a min-height for the windowdragbox (same as used in toolbars). We could skip that, since it will only have effect if someone does bad things with the tabs (content in the #tabs element and its padding/margin sizes the box to the correct height).

This patch (and the previous one) also contains some clean-up and the remaining tab style rules are now positioned straight after the only remaining #tabs rule. I also re-positioned and changed the size of the clearListButton (now it looks the same as the FF equivalent).
Attachment #551873 - Attachment is obsolete: true
Comment on attachment 551885 [details] [diff] [review]
v1.1

I will of course not land this until I have tested it on Lion (I should be able to do that soon).
Attachment #551885 - Flags: review?(mnyromyr)
Status: NEW → ASSIGNED
Fwiw, I tested the patch on Lion and I think it looks OK. The only thing that differ from the eq viewbuttons in Firefox's pageinfo window is that when you click a tab the background color of the tab will be much darker compared to when you click a viewbutton. I think we can live with that, though.
Attachment #551885 - Flags: review?(mnyromyr) → review+
http://hg.mozilla.org/comm-central/rev/53896297893a
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.8
You need to log in before you can comment on or make changes to this bug.