Closed
Bug 610688
Opened 14 years ago
Closed 14 years ago
Style buttons for popup notifications without secondary actions
Categories
(Toolkit :: Themes, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: Margaret, Assigned: Margaret)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
dao
:
review+
Dolske
:
approval2.0+
|
Details | Diff | Splinter Review |
Bug 610123 restored the default button styling to the buttons in popup notifications without secondary actions. However, we want these normal buttons to have the same updated style as the menu buttons.
Attachment #489189 -
Flags: review?(dao)
Comment 1•14 years ago
|
||
Comment on attachment 489189 [details] [diff] [review]
patch
This seems right for OS X.
Why would we want this for Windows? The reason we started at all with the custom menu button styling on Windows was that we didn't have a reasonable default style (bug 509642, bug 581193). This isn't the case for non-menu buttons.
Attachment #489189 -
Flags: review?(dao) → review-
Assignee | ||
Comment 2•14 years ago
|
||
(In reply to comment #1)
> Why would we want this for Windows? The reason we started at all with the
> custom menu button styling on Windows was that we didn't have a reasonable
> default style (bug 509642, bug 581193). This isn't the case for non-menu
> buttons.
I believe we want a custom button style for all arrow panels, but this is a first step. I talked to Stephen about it, and he's going to leave a comment once he decides what we should be doing.
Comment 3•14 years ago
|
||
(In reply to comment #2)
> (In reply to comment #1)
> > Why would we want this for Windows? The reason we started at all with the
> > custom menu button styling on Windows was that we didn't have a reasonable
> > default style (bug 509642, bug 581193). This isn't the case for non-menu
> > buttons.
>
> I believe we want a custom button style for all arrow panels,
Again, I don't see why. It looks and feels considerable worse than native buttons on Windows. (See the last part of bug 606343 comment 20 for the reaction of a random person.)
OS X is different since the panels are black there, so native buttons don't fit.
Assignee | ||
Comment 4•14 years ago
|
||
Here's a patch with only pinstripe styles. I'll file a separate bug if Stephen wants us to change the button styles on winstripe.
Attachment #489189 -
Attachment is obsolete: true
Attachment #489810 -
Flags: review?(dao)
Updated•14 years ago
|
Attachment #489810 -
Flags: review?(dao) → review+
Updated•14 years ago
|
OS: All → Mac OS X
Assignee | ||
Updated•14 years ago
|
Attachment #489810 -
Flags: approval2.0?
Comment 5•14 years ago
|
||
Comment on attachment 489810 [details] [diff] [review]
patch (pinstripe only)
a+=me, but let's get a followup on file to figure out what (if anything) we want to do on Windows... Sounds like shorlander has some thoughts.
Attachment #489810 -
Flags: approval2.0? → approval2.0+
Assignee | ||
Comment 6•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/898ef162e026
Filed bug 612690 as a follow-up.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•