Closed
Bug 595390
Opened 14 years ago
Closed 14 years ago
Don't use bold button labels
Categories
(Toolkit :: Application Update, defect)
Toolkit
Application Update
Tracking
()
RESOLVED
FIXED
mozilla2.0b7
People
(Reporter: micmon, Assigned: micmon)
Details
(Keywords: polish)
Attachments
(2 files, 1 obsolete file)
46.89 KB,
image/png
|
Details | |
2.97 KB,
patch
|
robert.strong.bugs
:
review+
mossop
:
approval2.0+
|
Details | Diff | Splinter Review |
The update manager displays some buttons with bold labels which is something I have not seen in any other Linux application.
Assignee | ||
Comment 1•14 years ago
|
||
Attachment #474261 -
Flags: ui-review?
Attachment #474261 -
Flags: review?
Comment 2•14 years ago
|
||
I had this same discussion with the Firefox UX team a while ago though it was about this not being normal for all platforms. I'll change the ui-review request to see if bold button text is still wanted.
Comment 3•14 years ago
|
||
Comment on attachment 474261 [details] [diff] [review] Remove bold rules removing review request until after ur-review.
Attachment #474261 -
Flags: ui-review?
Attachment #474261 -
Flags: review?
Comment 4•14 years ago
|
||
Changing summary with the following rationale. If user experience approves the ui-review the bold label should be removed at the very least on Windows since the default button is natively styled to draw attention to it on Windows. This isn't so clear on Mac OS X since our custom styling doesn't have native styling or custom styling to draw attention to it so it might be valuable to leave the text bold on Mac OS X.
OS: Linux → All
Hardware: x86 → All
Summary: Don't use bold button labels on Linux → Don't use bold button labels
Comment 5•14 years ago
|
||
Comment on attachment 474261 [details] [diff] [review] Remove bold rules Alex, this is in regards to the bold button text in the update UI that we've discussed previously. If it is decided to remove the bold text I would want it also removed at the very least on Windows for the reasons stated in comment #4.
Attachment #474261 -
Flags: ui-review?(faaborg)
Assignee | ||
Comment 6•14 years ago
|
||
As I originally filed this bug especially for Linux, here's a collection of screenshots showing the last pages of various wizard dialogs of Evolution (GNOME's official PIM app), Empathy (GNOME's official IM app) and OpenOffice Impress. The same results can be seen in Gimp and Ekiga for example.
Comment 7•14 years ago
|
||
Yep, the same is true for the other platforms and if we are going to change one they should be changed on all for the exact same reason you'd like it changed on Linux.
Updated•14 years ago
|
Attachment #474261 -
Flags: ui-review?(faaborg) → ui-review+
Comment 8•14 years ago
|
||
Comment on attachment 474261 [details] [diff] [review] Remove bold rules Awesome! Please remove the bold from all the themes... gnomestripe, pinstripe, and winstripe and I'll r+ it.
Attachment #474261 -
Flags: review-
Assignee | ||
Comment 9•14 years ago
|
||
Assignee: nobody → michael.monreal+moz
Attachment #474261 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #474489 -
Flags: review?(robert.bugzilla)
Updated•14 years ago
|
Attachment #474489 -
Flags: review?(robert.bugzilla)
Attachment #474489 -
Flags: review+
Attachment #474489 -
Flags: approval2.0?
Comment 10•14 years ago
|
||
I'll try to get approval and land this on Monday
Updated•14 years ago
|
Attachment #474489 -
Flags: approval2.0? → approval2.0+
Comment 11•14 years ago
|
||
Pushed to mozilla-central http://hg.mozilla.org/mozilla-central/rev/5ff46fc54245 Thanks for the patches!
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite-
Flags: in-litmus-
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b6
You need to log in
before you can comment on or make changes to this bug.
Description
•