Closed Bug 514829 Opened 11 years ago Closed 11 years ago

[gnomestripe] the old style folder icons are used in a number of places (those where folder.png is referred to)

Categories

(Thunderbird :: Mail Window Front End, defect)

All
Linux
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0b4

People

(Reporter: mkmelin, Assigned: mkmelin)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch proposed fix (obsolete) — Splinter Review
There are some places where the tb2 folder icons are still used. 

 - all move/copy to menus
 - the offline selection dialog (Click Advanced... in the imap synch settings)
 - imap/nntp subscription dialog
 - feed subscription dialog

Also, the filter dialog lacks icons totally for when selecting action target folders. (Requesting sr for this one-liner.)

Qute may need some of these fixes too, (especially msgSelectOffline.css should be easy to fix), but as i can't test it anyway, lets keep this to be about gnomestripe.

I notice archives (and outbox where applicable) special folder icons don't get set for the offline dialog, filter move to folder, although they are correct in folderMenus.css since it works in the move/copy menus. I'll file a new bug about that if there isn't one already.
Attachment #398837 - Flags: ui-review?(clarkbw)
Attachment #398837 - Flags: superreview?(bugzilla)
Attachment #398837 - Flags: review?(philringnalda)
Note to self: server.png should also go (in a separate bug), just the nntp server icons are used and should be moved to folder-pane.png

Which reminds me, the patch also fixes the look of saved searches with unseen messages. That was still using the old red star, that it actually worked out reasonably with the old folder icons peeking from behind seemed coincidental...
Flags: wanted-thunderbird3+
Target Milestone: --- → Thunderbird 3.0b4
Attached patch proposed fix, v2Splinter Review
Unbitrot myself
Attachment #398837 - Attachment is obsolete: true
Attachment #399308 - Flags: ui-review?(clarkbw)
Attachment #399308 - Flags: superreview?(bugzilla)
Attachment #399308 - Flags: review?(philringnalda)
Attachment #398837 - Flags: ui-review?(clarkbw)
Attachment #398837 - Flags: superreview?(bugzilla)
Attachment #398837 - Flags: review?(philringnalda)
(also hg remove mail/themes/gnomestripe/mail/icons/folder.png - forgot to re-include that, fixed locally)
Attachment #399308 - Flags: ui-review?(clarkbw) → ui-review+
Attachment #399308 - Flags: review?(philringnalda) → review+
Comment on attachment 399308 [details] [diff] [review]
proposed fix, v2

sr+a=Standard8.

I think this should be safe to land for b4 as well (mainly being theme based).
Attachment #399308 - Flags: superreview?(bugzilla)
Attachment #399308 - Flags: superreview+
Attachment #399308 - Flags: approval-thunderbird3+
changeset:   3602:806cadd87cb3
http://hg.mozilla.org/comm-central/rev/806cadd87cb3

->FIXED

Filed bug 515377 about the missing archives icon in the offline dialog.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.