Closed Bug 717881 Opened 9 years ago Closed 9 years ago

The default tab icon used on Mac doesn't exist

Categories

(Thunderbird :: Theme, defect)

x86
macOS
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 18.0

People

(Reporter: florian, Assigned: andreasn)

Details

Attachments

(1 file)

With a debug build I see this in my terminal:
Chrome file doesn't exist: [...]/obj-tbird/mozilla/dist/DailyDebug.app/Contents/MacOS/chrome/toolkit/skin/classic/global/tree/item.png

An MXR search shows that this file is referenced from:

http://mxr.mozilla.org/comm-central/source/mail/themes/pinstripe/mail/tabmail.css#48
48 .tabmail-tab {
49   list-style-image: url("chrome://global/skin/tree/item.png");
50 }

and from:
334 .alltabs-item {
335   list-style-image: url("chrome://global/skin/tree/item.png");
336 }

This item.png file doesn't exist.

This seems to cause the alltabs menu item for the inbox and email messages to not have any icon.
While the default icon should be fixed, we should probably also see why alltabs menuitem attempt to use it instead of the icon that these tabs have in the tab bar.
item.png was removed in bug 648668 (looks like it was replaced with chrome://mozapps/skin/places/defaultFavicon.png).
(In reply to Stefan [:stefanh] from comment #1)
> item.png was removed in bug 648668 (looks like it was replaced with
> chrome://mozapps/skin/places/defaultFavicon.png).

Oh, well - I don't think that defaultFavicon.png would be suitable here (just looked at it).
I was about to file a dup of this bug, mostly because I'm annoyed by the noise in the terminal, but it would be good to have some relevant icons displayed in the all tabs menu popup too. CC'ing Andreas
One cheap solution is to take the icon from 2.0 (http://mxr.mozilla.org/mozilla2.0/source/toolkit/themes/pinstripe/global/tree/item.png) and put it in mail/themes.
I haven't tested this, but I think it should do the trick.
I've just tested attachment 641047 [details] [diff] [review] and I can confirm that the warning in comment 0 no longer exists.

(In reply to Florian Quèze [:florian] [:flo] from comment #0)

> This item.png file doesn't exist.
> 
> This seems to cause the alltabs menu item for the inbox and email messages
> to not have any icon.

This is fixed by the patch here.

> While the default icon should be fixed, we should probably also see why
> alltabs menuitem attempt to use it instead of the icon that these tabs have
> in the tab bar.

I think we should fix this too, but we can handle it in another bug.

I was about to file a dup again for this today. Could we please get the fix reviewed and landed soon? :) (I almost put an r+ on the attachment after testing it and verifying that it works, but I'm not a peer for mail/themes/)
(In reply to Florian Quèze [:florian] [:flo] from comment #6)

> > While the default icon should be fixed, we should probably also see why
> > alltabs menuitem attempt to use it instead of the icon that these tabs have
> > in the tab bar.
> 
> I think we should fix this too, but we can handle it in another bug.

This has been fixed in bug 464524 :-).
Comment on attachment 641047 [details] [diff] [review]
patch to fix the issue

Setting up for review.
Attachment #641047 - Flags: review?(richard.marti)
Comment on attachment 641047 [details] [diff] [review]
patch to fix the issue

This looks good.
Attachment #641047 - Flags: review?(richard.marti) → review+
No checkin-needed ?
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/18d35f49e7fd
Assignee: nobody → nisses.mail
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
You need to log in before you can comment on or make changes to this bug.