polish tweaks for the header button box on mac

RESOLVED FIXED in Thunderbird 3.0rc1

Status

defect
P2
normal
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: andreasn, Assigned: andreasn)

Tracking

({polish})

Trunk
Thunderbird 3.0rc1
All
macOS
Bug Flags:
blocking-thunderbird3 +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [no l10n impact])

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

10 years ago
Two issues that was brought up on IRC:
* Forward icon have a different tint of blue that reply.
* The text color used in the button makes them look disabled.
The reply button is vertically a few pixels short.
Flags: blocking-thunderbird3?
Keywords: polish
Hardware: x86 → All
Target Milestone: --- → Thunderbird 3.0rc1
(Assignee)

Comment 2

10 years ago
I think if we are going to work on some new icons for the mac we should try some simplified versions of the current ones.  These buttons have a bit less vertical height than the toolbar buttons so I think we want to optimize for that by only showing the most necessary pieces.  I'm thinking of the Mail.app type icons (as they are similar sized buttons) where we are showing mostly a sweeping arrow without anything else around it.

Also it's likely that the toolbar buttons have some text-shadow happening to make them look more mac toolbar button like and that is creating this color difference as well.
(Assignee)

Comment 4

10 years ago
(Assignee)

Comment 5

10 years ago
Posted image proposed osx icons
Bryan: what about something like this?
Whiteboard: [no l10n impact]
I like those a lot, very simple and effective.  Lets try them out on the mac to see how they look in the buttons.
Flags: blocking-thunderbird3? → blocking-thunderbird3+
Priority: -- → P2
(Assignee)

Comment 7

10 years ago
Attachment #404026 - Attachment is obsolete: true
Attachment #405052 - Flags: ui-review?(clarkbw)
(Assignee)

Updated

10 years ago
Attachment #405052 - Flags: review?(dmose)
Comment on attachment 405052 [details] [diff] [review]
patch with new icons and text color fixed

r=dmose
Attachment #405052 - Flags: review?(dmose) → review+
Comment on attachment 405052 [details] [diff] [review]
patch with new icons and text color fixed

Comment 6 reads to me like it is ui-review+, so I'm going to mark it as such and land.
Attachment #405052 - Flags: ui-review?(clarkbw) → ui-review+
http://build.mozillamessaging.com/mercurial/comm-central/rev/f04e261ccca3
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.