Closed
Bug 388317
Opened 17 years ago
Closed 17 years ago
autocomplete and menulist popups should not be styled as native menus
Categories
(Toolkit :: Themes, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha8
People
(Reporter: zeniko, Assigned: zeniko)
References
()
Details
(Keywords: regression)
Attachments
(2 files)
965 bytes,
patch
|
enndeakin
:
review+
|
Details | Diff | Splinter Review |
869 bytes,
patch
|
enndeakin
:
review+
|
Details | Diff | Splinter Review |
Assignee | ||
Comment 1•17 years ago
|
||
If autocomplete popups got a native appearance, it wouldn't be menupopup to start with - but as it stands we can continue to style them completely through CSS.
Neil: Should all <panel>s really get a menupopup appearance (either native or through CSS) per default, anyway?
Updated•17 years ago
|
Attachment #272506 -
Flags: review?(enndeakin) → review+
Assignee | ||
Updated•17 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 2•17 years ago
|
||
So, there's another identical regression for menulist popups.
Summary: autocomplete popups are styled as menus, resulting in a white 2px border under Luna → autocomplete and menulist popups should not be styled as native menus
Assignee | ||
Comment 3•17 years ago
|
||
Attachment #273013 -
Flags: review?(enndeakin)
Updated•17 years ago
|
Attachment #273013 -
Flags: review?(enndeakin) → review+
Comment 4•17 years ago
|
||
mozilla/toolkit/themes/winstripe/global/popup.css 1.12 mozilla/toolkit/themes/winstripe/global/autocomplete.css 1.16
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9beta1
I don't know what this looks like on Windows, but it doesn't look very good on Linux. White, no borders, blends in with background of most pages... Needs a - border: 1px solid black; or some such.
Comment 6•17 years ago
|
||
Please see the testcase and screenshots in bug 388317, and in particular bug 388317 comment 4.
Comment 7•17 years ago
|
||
Damn, I was meaning bug 390711
Comment 8•17 years ago
|
||
I tried backing out this patch on a Windows trunk build (2007121405) and I don't see the gap between the selection and the border in the autocomplete panel (which can be seen on the screenshot posted on mozillazine).
Does somebody know what could have made this bug obsolete?
I'm asking because backing out this patch would solve bug 398242 (and remove the hack from bug 390711).
Comment 9•17 years ago
|
||
(In reply to comment #8)
> I'm asking because backing out this patch would solve bug 398242 (and remove
> the hack from bug 390711).
Well, that would not solve bug 398242 directly. If the policy of not using "moz-appearance: none" on panels for Windows reveals to be correct, the moz-appearance on the download info panel could be dropped for both Windows and Linux.
Comment 10•17 years ago
|
||
Ok, things are getting clearer. The changes on popup.css are still required, I could see the gap when removing the "-moz-appearance: none"
For the autocomple.css change, bug 355789 removed the "-moz-appearance: menupopup" on the panel element. This means setting the -moz-appearance to none explicitly in the autocomplete panel is not needed for preventing the gap between selection and border.
Neil, should I reopen this bug, remove the "-moz-appearance: none" and the #ifdef hack introduced in bug 390711?
Updated•16 years ago
|
Product: Core → SeaMonkey
Updated•15 years ago
|
Product: SeaMonkey → Toolkit
QA Contact: themes → themes
You need to log in
before you can comment on or make changes to this bug.
Description
•