82626 - Bad alignment of items in dropdown list

Status: VERIFIED FIXED

(Core :: Layout: Form Controls, defect)

Description

[Steve Green]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:0.9+) Gecko/20010524
BuildID:    2001052404

In the page at http://www.netbank.com/login.htm the text for the items in the
dropdown list ("select one...", "banking", "brokerage" "credit card" and "safe
deposit box") are misaligned.  When selected, only the bottom half of the
letters display.

Reproducible: Always
Steps to Reproduce:
1. Go to http://www.netbank.com/login.htm
2. Look at the dropdown list
3.

Comment 1

[Jason Johnston]

Confirmed on Linux-Mandrake8, build 2001052411

Comment 2

[Jason Johnston]

I'm thinking this may have to do with the fact that the dropdown in question is
within a <p> element, which has the CSS declaration:
p {line-height:0px;}

??? I don't know why anyone would want a line-height of 0px, but to each their
own...

The behavior still doesn't make sense to me though.

Comment 3

[rods (gone)]

Yes, that is it exactly. Somehow it must be inheriting that. If I add:
select {
line-height:1em ! important;
}
all is well.

Comment 4

[rods (gone)]

Attachment: reduced testcase

Comment 5

[rods (gone)]

Attachment: oops previous example has possible fix, this is the reduced testcase

Comment 6

[rods (gone)]

So the fix would be to add:

line-height:1em ! important;

to the forms.css file in the select rule.

Comment 7

[Vladimir Ermakov]

bug 82265 talks about the same issue

Comment 8

[Kevin McCluskey (gone)]

This bug could potentially make selects unusable because you wouldn't be able to
read whats in them.

patch looks good.
r=[e-mail]

Comment 9

[David Hyatt]

sr=hyatt

Comment 10

[Asa Dotzler [:asa]]

a= [e-mail] for checkin to the trunk.
(on behalf of drivers)

Comment 11

[rods (gone)]

fixed

Comment 12

[doctor__j]

Should we mark bug 79999 as a dupe of this one?

Comment 13

[rods (gone)]

Look like a round-off error accumulates, probably a core layout thing.

Comment 14

[rods (gone)]

this will have to wait

Comment 15

[Conor Lennon]

*** Bug 79999 has been marked as a duplicate of this bug. ***

Comment 16

[Boris Zbarsky [:bzbarsky]]

*** Bug 70019 has been marked as a duplicate of this bug. ***

Comment 17

[Boris Zbarsky [:bzbarsky]]

*** Bug 65187 has been marked as a duplicate of this bug. ***

Comment 18

[Boris Zbarsky [:bzbarsky]]

*** Bug 90149 has been marked as a duplicate of this bug. ***

Comment 19

[Aaron Kaluszka]

Did somebody apply this patch again?  Cause this bug seems to be gone, but Bug
84928 has returned.  WinXPRC2/2001081403-trunk

Comment 20

[Aaron Kaluszka]

oops looked at "fixed" testcase instead of real testcase, ignore my last comments

Comment 21

[rods (gone)]

*** Bug 97100 has been marked as a duplicate of this bug. ***

Comment 22

[Boris Zbarsky [:bzbarsky]]

*** Bug 105320 has been marked as a duplicate of this bug. ***

Comment 23

[rods (gone)]

Moving to Future

Comment 24

[rods (gone)]

*** Bug 117960 has been marked as a duplicate of this bug. ***

Comment 25

[TucsonTester3]

*** Bug 126761 has been marked as a duplicate of this bug. ***

Comment 26

[Vadim Berezniker]

*** Bug 139303 has been marked as a duplicate of this bug. ***

Comment 27

[R.K.Aa.]

*** Bug 143865 has been marked as a duplicate of this bug. ***

Comment 28

[Stewart Larsen]

*** Bug 149815 has been marked as a duplicate of this bug. ***

Comment 29

[Boris Zbarsky [:bzbarsky]]

*** Bug 150264 has been marked as a duplicate of this bug. ***

Comment 30

[Andreas Kunz]

Can someone please have a look at this? 
With 2002112208 on Win2k it looks correct (both testcase and URL) and the last
'positive' comment is more than a year old.

Comment 31

[Boris Zbarsky [:bzbarsky]]

This is fixed.  See bug 82265 for patch, discussion, etc.

Comment 32

[Vladimir Ermakov]

verifying build 2003-02-05-11-trunk