Closed
Bug 727598
Opened 12 years ago
Closed 12 years ago
Create icons for new Next and Previous message buttons created in bug 235666
Categories
(Thunderbird :: Theme, defect)
Thunderbird
Theme
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 16.0
People
(Reporter: aceman, Assigned: andreasn)
References
Details
(Keywords: polish)
Attachments
(5 files, 4 obsolete files)
15.27 KB,
image/png
|
Details | |
14.72 KB,
image/png
|
bwinton
:
ui-review+
|
Details |
212.84 KB,
patch
|
bwinton
:
ui-review-
aceman
:
feedback+
|
Details | Diff | Splinter Review |
16.39 KB,
image/png
|
bwinton
:
feedback+
andreasn
:
feedback+
|
Details |
216.93 KB,
patch
|
bwinton
:
review+
bwinton
:
ui-review+
aceman
:
feedback+
|
Details | Diff | Splinter Review |
This is a followup bug to bug 235666, where new buttons for Next (read) and Previous (read) message buttons were implemented. But they reuse the existing icons used for Next/Previous (Unread) message. So new icons should be created to differentiate the functions, as the button labels are intentionally identical. Currently thre next/previous icons contain 2 blue arrows (at least in gnomestripe theme). Maybe the same design with 1 arrow could be drawn. This bug needs somebody who knows to create those icons and land them into the theme files (mail-toolbar.png and mail-toolbar-small.png). After that I can make the necessary css updates in the theme to make the buttons use the new icons.
Anybody working on this could also note bug 243480. Maybe all the 4 icons could be updated.
Depends on: 243480
Andreas, I got a hint you could be the right person for this.
Keywords: uiwanted
Assignee | ||
Comment 3•12 years ago
|
||
(In reply to :aceman from comment #2) > Andreas, I got a hint you could be the right person for this. Yes, new icons coming up!
Assignee | ||
Comment 4•12 years ago
|
||
I'll upload a variant that takes bug 243480 into account as well shortly.
Bwinton, do you actually agree with bug 243480?
Andreas, I do not understand the proposal. Which are the new icons in the image?
Comment 7•12 years ago
|
||
Yeah, the selection moves up and down, not left and right, so having previous and next go that way makes sense to me, too.
Assignee | ||
Comment 8•12 years ago
|
||
And this version with the arrows pointing down. It's the two icons to the very right (the regular next&previous icons are also fixed).
Assignee | ||
Comment 10•12 years ago
|
||
it's the same icon we use for "read"
Assignee | ||
Comment 11•12 years ago
|
||
if these are ok I'll do qute and gnomestripe as well.
Reporter | ||
Comment 12•12 years ago
|
||
Comment on attachment 627880 [details]
pinstripe mail-toolbar proposal 2
I do not have much opinion on how they should look like. I take anything.
I can only test the gnomestripe version.
Attachment #627880 -
Flags: ui-review?(bwinton)
Comment 13•12 years ago
|
||
Comment on attachment 627880 [details]
pinstripe mail-toolbar proposal 2
Yeah, I like those… ui-r=me!
Thanks,
Blake.
Comment 14•12 years ago
|
||
Comment on attachment 627880 [details]
pinstripe mail-toolbar proposal 2
(I would be even cooler if I actually remembered to change the ui-review flag to + when I gave my ui-r+… ;)
Attachment #627880 -
Flags: ui-review?(bwinton) → ui-review+
Assignee | ||
Comment 15•12 years ago
|
||
Still needs css.
Reporter | ||
Comment 16•12 years ago
|
||
I can do the css if you wish.
Assignee | ||
Comment 17•12 years ago
|
||
Need to actually try it in action on all platforms first, but I think this should do the trick.
Attachment #630189 -
Attachment is obsolete: true
Assignee | ||
Comment 18•12 years ago
|
||
Had accidentally switched the numbers around for unread and regular next/previous. This also takes out the rtl variants, as the graphics now goes up and down.
Attachment #631004 -
Attachment is obsolete: true
Reporter | ||
Comment 19•12 years ago
|
||
Thanks, I'll try it out.
Assignee: nobody → nisses.mail
Status: NEW → ASSIGNED
Reporter | ||
Comment 20•12 years ago
|
||
Comment on attachment 631406 [details] [diff] [review] icons+css v2 Thanks! I can confirm the new icons work in Linux (gnomestripe), LTR setup, both normal and small toolbar and standalone message window.
Attachment #631406 -
Flags: feedback+
Assignee | ||
Comment 21•12 years ago
|
||
Comment on attachment 631406 [details] [diff] [review] icons+css v2 Setting up for review
Attachment #631406 -
Flags: ui-review?(bwinton)
Attachment #631406 -
Flags: review?(mconley)
Comment 22•12 years ago
|
||
Comment on attachment 631406 [details] [diff] [review] icons+css v2 Okay, I know this is totally a dick-move, but when I see the next-unread/previous-unread, I'm not really happy with the sunglasses (at least on OS X). We aren't using them in the QFB, or in the message-list-header, so they seem odd and out-of-place. What do you think about a double-up-arrow/double-down-arrow instead? Or making them the same as the next/previous icons, like we do now? Sorry about that, Blake.
Attachment #631406 -
Flags: ui-review?(bwinton) → ui-review-
Reporter | ||
Comment 23•12 years ago
|
||
They should not be the same, the point of this bug is to distinguish them. But making the prev/next be 1 arrow, and prev/next unread be 2 arrows was in the original proposal :)
Comment 24•12 years ago
|
||
Yeah, I'm really sorry about that. :(
Comment 25•12 years ago
|
||
Comment on attachment 631406 [details] [diff] [review] icons+css v2 Removing r? until the icons issue is resolved.
Attachment #631406 -
Flags: review?(mconley)
Reporter | ||
Comment 26•12 years ago
|
||
So here is my proposal, based on gnomestripe theme. Rotate existing prev/next unread icons (double tip-of-an-arrow) and add new icons for prev/next read action (single tip-of-an-arrow). Other themes do not seem to use the white sheet behind the arrows, so adapt each theme as necessary (actually, copy existing double arrow and make single arrow version, and rotate both). This is just a mockup, please align the arrows nicely in the final icons (so they can't be just 90deg rotations, there need to be some adjustments).
Attachment #636439 -
Flags: feedback?(bwinton)
Attachment #636439 -
Flags: feedback?(nisses.mail)
Comment 27•12 years ago
|
||
Comment on attachment 636439 [details]
icons proposal
I think this works for me. I'ld like to see what it would look like on the other platforms, too, but I understand if you can't do that, aceman…
Thanks,
Blake.
Attachment #636439 -
Flags: feedback?(bwinton) → feedback+
Reporter | ||
Comment 28•12 years ago
|
||
I could do it, but it is mostly the same. I just wanted to show both of you how the directions and double/single arrows are meant.
Assignee | ||
Comment 29•12 years ago
|
||
Comment on attachment 636439 [details]
icons proposal
I'm planning on dropping the sheets behind those icons, but apart from that, I agree with this plan.
Attachment #636439 -
Flags: feedback?(nisses.mail) → feedback+
Assignee | ||
Comment 30•12 years ago
|
||
Here are the new icons, replacing the files that bitrotted with the landing of #748358. Updated css coming shortly.
Assignee | ||
Comment 31•12 years ago
|
||
Same as above, with some css. Badly formatted right now.
Attachment #636725 -
Attachment is obsolete: true
Reporter | ||
Comment 32•12 years ago
|
||
Comment on attachment 636784 [details] [diff] [review] new icons, new css Why did the css need change? Wasn't the previous version fine just the icons needed replacement?
Assignee | ||
Comment 33•12 years ago
|
||
It's the same css, just not bitrotted.
Reporter | ||
Comment 34•12 years ago
|
||
Comment on attachment 636784 [details] [diff] [review] new icons, new css Great, just tested the patch on gnomestripe and the icons are nice, as I proposed. Only I think you got them swapped in the css. The double arrow now does "next/prev read" instead of "next/prev unread" as before.
Attachment #636784 -
Flags: feedback-
Assignee | ||
Comment 35•12 years ago
|
||
Attachment #636784 -
Attachment is obsolete: true
Attachment #637068 -
Flags: review?(acelists)
Reporter | ||
Comment 36•12 years ago
|
||
Comment on attachment 637068 [details] [diff] [review] new icons and css (v3) Thanks, this is what I had in mind.
Attachment #637068 -
Flags: ui-review?(bwinton)
Attachment #637068 -
Flags: review?(acelists)
Attachment #637068 -
Flags: feedback+
Comment 37•12 years ago
|
||
Comment on attachment 637068 [details] [diff] [review] new icons and css (v3) Awesome! ui-r=me! Later, Blake.
Attachment #637068 -
Flags: ui-review?(bwinton) → ui-review+
Reporter | ||
Comment 38•12 years ago
|
||
The patch is just css and icon files. Who can review that?
Comment 39•12 years ago
|
||
Comment on attachment 637068 [details] [diff] [review] new icons and css (v3) (In reply to :aceman from comment #38) > The patch is just css and icon files. Who can review that? Good question. I can. r=me. :) Later, Blake.
Attachment #637068 -
Flags: review+
Keywords: checkin-needed
Comment 40•12 years ago
|
||
https://hg.mozilla.org/comm-central/rev/bcd212ed28fc
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 16.0
Comment 41•12 years ago
|
||
In the default theme for Thunderbird 16.0.2 on Ubuntu 12.10, the next arrow points up until you put the mouse over it. Many themes that work well in the Linux such as Silvermel still do not have Previous and Next icons. Oxygenate, the perfect theme while using the KDE manager, is no longer compatible. Is there a way for all themes to fall back to a default icon if it is not defined in a theme? If there is, I think that this should be a requirement for all themes uploaded to Mozilla Add-ons. Themes, just as extensions, available at other websites would be downloaded and installed at the risk of the user for quality, functionality, and malicious code.
Reporter | ||
Comment 42•12 years ago
|
||
The Ubuntu icon problem is bug 801514. Ubuntu uses a different theme than Thunderbird ships for Linux.
You need to log in
before you can comment on or make changes to this bug.
Description
•