Closed
Bug 512548
Opened 15 years ago
Closed 15 years ago
[Mac Classic] Use "pressed" look for toolbarbuttons when their menu is open
Categories
(SeaMonkey :: Themes, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.0
People
(Reporter: stefanh, Assigned: stefanh)
References
Details
(Keywords: classic, fixed-seamonkey2.0)
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
neil
:
superreview+
kairo
:
approval-seamonkey2.0+
|
Details | Diff | Splinter Review |
This is the SeaMonkey version of bug 510808.
Assignee | ||
Comment 1•15 years ago
|
||
I hope you don't mind, but I couldn't resist fixing those whitespaces.
Attachment #396548 -
Flags: review?(mnyromyr)
Assignee | ||
Updated•15 years ago
|
Target Milestone: --- → seamonkey2.0b2
Updated•15 years ago
|
Attachment #396548 -
Flags: review?(mnyromyr) → review+
Assignee | ||
Updated•15 years ago
|
Attachment #396548 -
Flags: superreview?(neil)
Comment 2•15 years ago
|
||
Comment on attachment 396548 [details] [diff] [review]
Fix toolbarbuttons
>-#printButton:hover:active {
>+#printButton:hover:active,
>+#printButton[open] {
All these changes look reasonable.
> toolbar[iconsize="small"] > toolbarpaletteitem > #back-button:hover:active,
>-toolbar[iconsize="small"] > #back-button:hover:active {
>+toolbar[iconsize="small"] > toolbarpaletteitem > #back-button[open],
>+toolbar[iconsize="small"] > #back-button:hover:active,
>+toolbar[iconsize="small"] > #back-button[open] {
But I don't understand all these changes, since you shouldn't be able to have an active, disabled or open toolbarpaletteitem child... (a toolbarpaletteitem is a dynamic node created during toolbar customisation)
Assignee | ||
Comment 3•15 years ago
|
||
I reverted my changes that affects stuff in customizing mode - I also removed the disabled buttons style rules. Actually, it'd be nicer if they worked, but that feels like another bug.
Attachment #396548 -
Attachment is obsolete: true
Attachment #397362 -
Flags: superreview?(neil)
Attachment #396548 -
Flags: superreview?(neil)
Assignee | ||
Comment 4•15 years ago
|
||
Filed bug 512548 for non-mac/Modern removal of the redundant style rules.
Assignee | ||
Updated•15 years ago
|
Attachment #397362 -
Flags: superreview?(neil)
Assignee | ||
Comment 5•15 years ago
|
||
Comment on attachment 397362 [details] [diff] [review]
New version
I just realized that I forgot the help buttons...
Assignee | ||
Comment 6•15 years ago
|
||
Attachment #397362 -
Attachment is obsolete: true
Attachment #397687 -
Flags: superreview?(neil)
Assignee | ||
Comment 7•15 years ago
|
||
(In reply to comment #4)
> Filed bug 512548 for non-mac/Modern removal of the redundant style rules.
Erm, bug 513453.
Comment 8•15 years ago
|
||
Comment on attachment 397687 [details] [diff] [review]
Also fix helpOverlay.css
>+#help-back-button:not([disabled="true"])[open] {
This is incorrect for a number of reasons:
1. :not rules belong after other rules
2. #help-back-button[disabled="true"] naturally overrides #help-back-button[open] (but not #help-back-button:hover:active)
3. It's not possible to open disabled buttons
So the rule should just be for #help-back-button[open] (and similarly for the other buttons). sr=me with that fixed.
Attachment #397687 -
Flags: superreview?(neil) → superreview+
Assignee | ||
Comment 9•15 years ago
|
||
JFTR: as discussed on irc, I'll keep the patch as is since (even though it sucks) not using the :not would mean that the preceeding :hover rule will be more specific.
Comment 10•15 years ago
|
||
You can at least put the :not at the end right?
Assignee | ||
Comment 11•15 years ago
|
||
Comment on attachment 397687 [details] [diff] [review]
Also fix helpOverlay.css
Additional mac look and feel enhancement + some css cleanup.
Attachment #397687 -
Flags: approval-seamonkey2.0?
Updated•15 years ago
|
Attachment #397687 -
Flags: approval-seamonkey2.0? → approval-seamonkey2.0+
Assignee | ||
Comment 12•15 years ago
|
||
http://hg.mozilla.org/comm-central/rev/c53d94c4ef0b (with comment #10 addressed).
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: seamonkey2.0b2 → seamonkey2.0
Comment 13•15 years ago
|
||
Adding fixed-seamonkey2.0 keyword so the queries for approved but not fixed bugs don't catch that bug.
Keywords: fixed-seamonkey2.0
You need to log in
before you can comment on or make changes to this bug.
Description
•