Closed
Bug 14972
Opened 25 years ago
Closed 25 years ago
Toolbar popup menu hover
Categories
(Core :: XUL, defect, P3)
Tracking
()
VERIFIED
FIXED
M16
People
(Reporter: michael.j.lowe, Assigned: bugs)
Details
Popup menus on the toolbar should be drawn with the same outline hover style as
plain toolbar buttons. At present popup menus on the toolbar don't display any
hover style when you mouse over them.
Updated•25 years ago
|
QA Contact: beppe → cpratt
Updated•25 years ago
|
Assignee: trudelle → hangas
Comment 1•25 years ago
|
||
reassigning to hangas per hyatt, can you verify this is still happening with the
new UI?
Reporter | ||
Comment 2•25 years ago
|
||
This bug can be fixed by adding these rules to skin.css:
toolbar > menu:hover > titledbutton {
border: 1px outset white;
color:#333366;
text-decoration: underline;
}
toolbar > menu[open=true] > titledbutton {
border: 1px inset white;
}
Including German to help understand what needs to be done here. I suspect that
the new skin handles the hover case in a different way.
Reporter | ||
Comment 5•25 years ago
|
||
Yes, the problem is still there but my suggested fix will need to be modified
for the new skin style. Have a look at the Composer. On the toolbar, moving
the mouse over the Font or Size popup menus on the toolbar does not display the
same feedback as the other buttons on the toolbar. Also, opening the popup
menu should display the same open feedback as what normal menus have. This is
what this change aims to fix.
German, we should talk this over and make sure we agree on a default behavior for
menus in toolbars.
I think the problem is that in the new skin the buttons need to be assigned both
classes "push" and "popup" in order to get the proper behavior, such like class=
"push popup". This might solve the problem
Comment 9•25 years ago
|
||
Is this a beta stopper?
Assignee | ||
Comment 12•25 years ago
|
||
looks fixed to me.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Comment 14•25 years ago
|
||
VERIFIED with 2000032809 builds on all platforms
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•