Closed Bug 610688 Opened 14 years ago Closed 14 years ago

Style buttons for popup notifications without secondary actions

Categories

(Toolkit :: Themes, defect)

All
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: Margaret, Assigned: Margaret)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch patch (obsolete) (deleted) — 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 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-
(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.
(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.
Attached patch patch (pinstripe only) (deleted) — Splinter Review
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)
Attachment #489810 - Flags: review?(dao) → review+
OS: All → Mac OS X
Attachment #489810 - Flags: approval2.0?
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+
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.

Attachment

General

Created:
Updated:
Size: