Closed
Bug 348173
Opened 18 years ago
Closed 17 years ago
Menu popups missing a border in non-flat style
Categories
(SeaMonkey :: Themes, defect)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: zeniko, Unassigned)
References
Details
Attachments
(2 files)
16.81 KB,
image/png
|
Details | |
1.23 KB,
patch
|
asaf
:
review+
|
Details | Diff | Splinter Review |
In the spirit of 313388, menu popups should be modified in the following way to better match Windows Classic appearance. Note the third color instead of the 1px padding which makes a minor difference when Menu (background) != ThreeDFace. border: 3px solid; -moz-border-top-colors: ThreeDLightShadow ThreeDHighlight ThreeDFace; -moz-border-right-colors: ThreeDDarkShadow ThreeDShadow ThreeDFace; -moz-border-bottom-colors: ThreeDDarkShadow ThreeDShadow ThreeDFace; -moz-border-left-colors: ThreeDLightShadow ThreeDHighlight ThreeDFace; padding: 0px;
Reporter | ||
Comment 1•18 years ago
|
||
Updated•18 years ago
|
Assignee: nobody → silver
Updated•18 years ago
|
Status: NEW → ASSIGNED
Comment 2•18 years ago
|
||
Simon, well caught with this bug. :)
Attachment #233463 -
Flags: review?(kevin)
Updated•18 years ago
|
Updated•18 years ago
|
Assignee: silver → nobody
Status: ASSIGNED → NEW
Reporter | ||
Comment 3•18 years ago
|
||
Comment on attachment 233463 [details] [diff] [review] Use 3D-face colour for "padding" I somehow doubt that Kevin is going to have a look at this patch anytime soon...
Attachment #233463 -
Flags: review?(kevin) → review?(mano)
Comment 4•18 years ago
|
||
Comment on attachment 233463 [details] [diff] [review] Use 3D-face colour for "padding" ask beltzner for ui-r before checkin.
Attachment #233463 -
Flags: review?(mano) → review+
Reporter | ||
Comment 5•18 years ago
|
||
Comment on attachment 233463 [details] [diff] [review] Use 3D-face colour for "padding" Mike: See attachment 233054 [details] for a comparison of what we've got and what it should look like with this patch. Asking for sr since there's no ui-r in this component.
Attachment #233463 -
Flags: superreview?(beltzner)
Reporter | ||
Comment 6•17 years ago
|
||
Mike: This is another case which should be quite obvious. Any chance you can spare five minutes on it?
Keywords: uiwanted
Reporter | ||
Updated•17 years ago
|
Attachment #233463 -
Flags: superreview?(beltzner)
Reporter | ||
Comment 7•17 years ago
|
||
This bug's been fixed with the check-in of bug 337771 -> WORKSFORME.
Assignee | ||
Updated•16 years ago
|
Product: Core → SeaMonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•