Closed
Bug 683786
Opened 13 years ago
Closed 13 years ago
checkbox missing in dropdowns in web pages when tablet ui is turned on
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 9
People
(Reporter: nhirata, Assigned: sriram)
References
()
Details
(Keywords: regression)
Attachments
(2 files)
66.85 KB,
image/png
|
Details | |
1.72 KB,
patch
|
mbrubeck
:
review+
|
Details | Diff | Splinter Review |
1. turn on tablet ui 2. go to http://people.mozilla.com/~nhirata/html_tp/formsninput.html 3. select the dropdown Expected: to see a checkmark next to the current selection Actual: nothing Mozilla/5.0 (Android; Linux armv7I; rv9.0a1) Gecko/20110831 Firefox/9.0a1 Fennec/9.0a1 Device: Toshiba Thrive OS Android 3.1
Reporter | ||
Updated•13 years ago
|
OS: Mac OS X → Android
Hardware: x86 → ARM
Comment 1•13 years ago
|
||
Sriram, can you take a look at this please?
Assignee | ||
Comment 2•13 years ago
|
||
Here is my analysis: 1. There is no form.css inside /themes/core/honeycomb/ - which makes it to take the default css. 2. The default css uses check-30.png for the check mark. This is not available inside honeycomb folder, and there is no define in jar.mn - this makes it not to display a checkmark. I guess, we would have to wait for the form controls to be flushed out by Ian Barlow for honeycomb theme, to come up with forms.css. I would be happy to work on forms.css for honeycomb :)
Comment 3•13 years ago
|
||
For the moment, just to stay in a more or less usable state, let's either add back honeycomb/images/check-30.png, or add a honeycomb/forms.css that uses one of the current honeycomb checkmark images. Whichever works best. (Either way, we can change it again once we get real specs from Ian.)
Assignee | ||
Comment 4•13 years ago
|
||
This is a temporary patch until the honeycomb theme is flushed out completely. I changed the jar.mn to make the honeycomb theme refer to froyo's check mark for the select's pop-up box.
Attachment #557564 -
Flags: review?(mbrubeck)
Updated•13 years ago
|
Attachment #557564 -
Flags: review?(mbrubeck) → review+
Comment 5•13 years ago
|
||
hg.mozilla.org/integration/mozilla-inbound/rev/4f3eb6d6dcf5
Status: NEW → ASSIGNED
Whiteboard: [inbound]
Comment 6•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/4f3eb6d6dcf5
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → Firefox 9
Comment 7•13 years ago
|
||
Mozilla/5.0 (Android, Linux armv7l; rv:9.0a1) Gecko/20110906 Firefox/9.0a1 Fennec/9.0a1
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•