Closed Bug 1095904 Opened 10 years ago Closed 7 years ago

[Mac Classic] Style left panes in Bookmarks Manager, MailNews and Address Book as source lists

Categories

(SeaMonkey :: Themes, defect)

SeaMonkey 2.54 Branch
defect
Not set
normal

Tracking

(seamonkey2.49esr fixed, seamonkey2.50 wontfix, seamonkey2.51 wontfix, seamonkey2.52 wontfix, seamonkey2.54 fixed, seamonkey2.53 fixed)

RESOLVED FIXED
seamonkey2.54
Tracking Status
seamonkey2.49esr --- fixed
seamonkey2.50 --- wontfix
seamonkey2.51 --- wontfix
seamonkey2.52 --- wontfix
seamonkey2.54 --- fixed
seamonkey2.53 --- fixed

People

(Reporter: stefanh, Assigned: stefanh)

References

Details

Attachments

(1 file, 1 obsolete file)

Tree rows are currently styled with the standard colors from tree.css, but tree rows in the Bookmarks Manager, MailNews and Address Book sidebars should be styled differently (like in Finder).
Attached patch WIP (obsolete) — Splinter Review
I decided to use a class for those 3 trees and style them in communicator.css instead of having 3 files to maintain. Should probably do the same with the splitter... This looks OK, I'm just a bit disturbed by the background-color of the tree header since it conflicts a bit the blue gradient of selected tree rows. Interestingly, the background-color of the tree headers (list views) look slightly different in native mac apps... it's not as soft as ours.
Summary: [Mac Classic] Finder-like tree rows in Bookmarks Manager, MailNews and Address Book sidebars → [Mac Classic] Style left panes in Bookmarks Manager, MailNews and Address Book as source lists
OK, so this is the final version. Compared to the previous patch i haven't used used any additional classes - I just use what we currently have which means some code duplication.

When it comes to mailnews and address book, I don't change the font-weight like in BM - the reason for that is of course that it would mess up the usage of 'bold' as a symbol of new mail and account name. I realize now that I could chnage the font-size in address book - tell me if you think that would be appropiate (but maybe it's a bit inconsistent?). Think about it, atm I'm having trouble deciding what route to take here :-)

The new style require a inverted twisty and we currently don't have any means of getting that with native styling. The use of an svg image creates a side-effect: bug 1280812, which I work-around by setting a hard-coded width of the twisty (that's the "width: 16px;" in 3 places).
Attachment #8519583 - Attachment is obsolete: true
Attachment #8896636 - Flags: review?(frgrahl)
Status: NEW → ASSIGNED
Hmm, I guess I could also skip the font-weight changes in the BM manager...
Comment on attachment 8896636 [details] [diff] [review]
[Mac Classic] Style left panes in Bookmarks Manager, MailNews and Address Book as source lists

> Think about it, atm I'm having trouble deciding what route to take here :-)

All 3 look a little different when selected. For me under 10.12.6 the variant in the mail window is the most readable and matches Finder most closely. I would change all 3 to be consistent. Pick your favorite :)

Twisties looks fine to me.

r+ with selection made consistent.
Attachment #8896636 - Flags: review?(frgrahl) → review+
There's no difference between Address Book and MailNews - maybe you had the window over another background when checking? Amyway, give me a few days to think about this (there's also 10.9, which looks completely different - the selected bg color is a blue gradient there and there's no vibrancy at all) ;-)
> There's no difference between Address Book and MailNews

There is a nice Thunderstorm brewing here right now. Will recheck it later. Thought the font colow when selected was different.

FRG
> There's no difference between Address Book and MailNews - maybe you had the window over another background when checking? 

I tried again and the colors seem to be more muted in the address book even when the background is the same. Same for selection. Hard to say. Overall you are right. Nothing to be worth investigating. Or I might just need to need new glasses. Probably an illusion caused by the different words in the selection panes.
Pushed by stefanh@inbox.com:
https://hg.mozilla.org/comm-central/rev/dfa8c31dafd8
[Mac Classic] Style left panes in Bookmarks Manager, MailNews and Address Book as source lists. r=frg.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
I took the route of making the BM manager behave the same as the rest. Will wait a day or two before asking approval for c-b and comm-esr52.
Version: Trunk → SeaMonkey 2.54 Branch
> Will wait a day or two before asking approval for c-b and comm-esr52.

Just do it now imho. 2.54 is in bad shape right now and no one will probably test it. When it is in comm-esr52 we can set the final csets for 2.49.1
Target Milestone: --- → seamonkey2.54
Comment on attachment 8896636 [details] [diff] [review]
[Mac Classic] Style left panes in Bookmarks Manager, MailNews and Address Book as source lists

(In reply to Frank-Rainer Grahl (:frg) from comment #11)
> > Will wait a day or two before asking approval for c-b and comm-esr52.
> 
> Just do it now imho. 2.54 is in bad shape right now and no one will probably
> test it. When it is in comm-esr52 we can set the final csets for 2.49.1

Right, I don't see any recent Mac builds in https://archive.mozilla.org/pub/seamonkey/nightly/latest-comm-central-trunk/... I've actually tested this more on comm-esr52 (running a comm-esr52 build with the patch right now).


[Approval Request Comment]
Risk to taking this patch (and alternatives if risky): Low, CSS-changes only.
String changes made by this patch: None.
Attachment #8896636 - Flags: approval-comm-esr52?
Attachment #8896636 - Flags: approval-comm-beta?
Summary: [Mac Classic] Style left panes in Bookmarks Manager, MailNews and Address Book as source lists → [Mac default] Style left panes in Bookmarks Manager, MailNews and Address Book as source lists
> Right, I don't see any recent Mac builds 

There is only one builder left which is overwhelmed. The others play with the beachbunnies according to ewong. Tinderbox builds were still done last week.

I only tested on comm-esr52 so this should be a no brainer.
Summary: [Mac default] Style left panes in Bookmarks Manager, MailNews and Address Book as source lists → [Mac Classic] Style left panes in Bookmarks Manager, MailNews and Address Book as source lists
Comment on attachment 8896636 [details] [diff] [review]
[Mac Classic] Style left panes in Bookmarks Manager, MailNews and Address Book as source lists

a=me, thanks
Attachment #8896636 - Flags: approval-comm-esr52?
Attachment #8896636 - Flags: approval-comm-esr52+
Attachment #8896636 - Flags: approval-comm-beta?
Attachment #8896636 - Flags: approval-comm-beta+
You need to log in before you can comment on or make changes to this bug.