539128 - simple reply, reply all, and forward icons

Status: RESOLVED WONTFIX

(Thunderbird :: Message Reader UI, defect)

Description

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

This is a request for Windows as the Mac version of Thunderbird has new reply, reply all and forward icons it is using in the header area which are much simpler and cleaner.

I think we want to use a similar color scheme as we already have for windows but remove the mail and people pieces so we just have arrows representing the directions.

Assigning to Andreas

Comment 1

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

marking wanted for the 3.0.x releases

Comment 2

[Andreas Nilsson (:andreasn)]

Attachment: patch for qute (xp/aero) and gnomestripe

Comment 3

[Andreas Nilsson (:andreasn)]

Attachment: vista screenshot

Comment 4

[Andreas Nilsson (:andreasn)]

Attachment: small revisions to the xp/aero graphics

and some css added for things to actually work in gnomestripe :)

Comment 5

[Andreas Nilsson (:andreasn)]

Attachment: gnomestripe screenshot

Comment 6

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

Comment on attachment 421798 [details] [diff] [review]
small revisions to the xp/aero graphics

cool!

Comment 7

[Andreas Nilsson (:andreasn)]

Comment on attachment 421798 [details] [diff] [review]
small revisions to the xp/aero graphics

on Mark Banners suggestion, I'm setting Blake Winton as reviewer instead, as Dan have a lot of stuff on his table.

Comment 8

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 421798 [details] [diff] [review]
small revisions to the xp/aero graphics

>+++ b/mail/themes/gnomestripe/mail/messageHeader.css
>@@ -247,18 +247,18 @@
>-  list-style-image: url("chrome://messenger/skin/icons/mail-toolbar-small.png");
>+  list-style-image: url("chrome://messenger/skin/icons/reply-arrows.png");

If we're moving the arrows out of the toolbar, does it make sense to edit the toolbar image to not include them anymore?

Other than that, r=me.

Later,
Blake.

Comment 9

[Mark Banner (:standard8)]

Not sure this is wanted for 3.0 any more, I'm generally nervous about theme updates on a stable branch. Would be good to get an updated patch for 3.1 though.

Comment 10

[Andreas Nilsson (:andreasn)]

Attachment: un-bitrotted patch

Here is a unbitrotted patch. The file seems to refer to bigger sizes as well, but I'm not sure how to trigger those for the messageheader.

Comment 11

[Andreas Nilsson (:andreasn)]

Bryan: do you know why we have the bigger sizes in there as well?

Comment 12

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

(In reply to comment #11)
> Bryan: do you know why we have the bigger sizes in there as well?

I believe that's for the toolbar when you uncheck the [x] use small icons item in the customize palette.

Comment 13

[:aceman]

The patch has review.
Was it already checked in or why is it stale?

Comment 14

[Andreas Nilsson (:andreasn)]

These issues have actually been solved elsewhere.
The Windows icons have been fixed in bug #545557 and the Linux part is bug #666229
I think WONTFIX is the right flag here.