Closed
Bug 323040
Opened 19 years ago
Closed 19 years ago
RTL <select>s have incorrect horizontal padding
Categories
(Core :: Layout: Form Controls, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha1
People
(Reporter: uriber, Assigned: uriber)
Details
(Keywords: fixed1.8.1, rtl)
Attachments
(2 files)
|
(deleted),
patch
|
dbaron
:
review+
dbaron
:
superreview+
dbaron
:
approval-branch-1.8.1+
|
Details | Diff | Splinter Review |
|
(deleted),
image/png
|
Details |
screenshot
The horizontal paddings inside <select> elements are currently specified in absolute terms (left vs. right). As a result, there is not enough right padding on the text in a closed "combobox" select (some letters are even partially obscured), and there is too much right padding on options in a "multiline" select.
Horizontal paddings should be specified using -moz-padding-start and -moz-padding-end.
|
Assignee | |
Comment 1•19 years ago
|
||
Assignee: nobody → uriber
Status: NEW → ASSIGNED
Attachment #208198 -
Flags: superreview?(dbaron)
Attachment #208198 -
Flags: review?(dbaron)
|
|
Assignee | |
Comment 2•19 years ago
|
||
before (top) and after (bottom).
|
||
Updated•19 years ago
|
Attachment #208198 -
Flags: superreview?(dbaron)
Attachment #208198 -
Flags: superreview+
Attachment #208198 -
Flags: review?(dbaron)
Attachment #208198 -
Flags: review+
|
|
Assignee | |
Updated•19 years ago
|
Attachment #208198 -
Flags: approval1.8.1?
|
|
Assignee | |
Comment 3•19 years ago
|
||
Checked in:
Checking in layout/style/forms.css;
/cvsroot/mozilla/layout/style/forms.css,v <-- forms.css
new revision: 3.112; previous revision: 3.111
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9alpha
|
|
Assignee | |
Updated•19 years ago
|
Attachment #208198 -
Flags: approval1.8.1? → branch-1.8.1?(dbaron)
|
|
||
Updated•19 years ago
|
Attachment #208198 -
Flags: branch-1.8.1?(dbaron) → branch-1.8.1+
|
|
Assignee | |
Comment 4•19 years ago
|
||
Checked in to 1.8.1 branch:
/cvsroot/mozilla/layout/style/forms.css,v <-- forms.css
new revision: 3.104.4.5; previous revision: 3.104.4.4
Keywords: fixed1.8.1
|
||
Comment 5•17 years ago
|
||
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
You need to log in
before you can comment on or make changes to this bug.
Description
•