Closed Bug 155483 Opened 18 years ago Closed 18 years ago
line-height does not work for input
We are adding the testcase. Note that the simple testcase have a INPUT TEXT element and a SELECT. Both are binded with the TEST CSS class. The SELECT field works okay with line-height but that does not work with the INPUT.
-> form controls (not that I think it should work, but...)
Assignee: attinasi → rods
Component: Layout → HTML Form Controls
QA Contact: petersen → tpreston
Summary: line-height does not work for input. → line-height does not work for input.
We are also looking for workarounds to this case. This happes with TopSite in China.
This new variation of the testcase demonstrates that the line-height sort of works with the input element. I added height:60px to the input in the inline style definition. Note that this have "Activated" the line-height of the input.
I believe that form elements, which are replaced, should not be affected by the property 'line-height', so the bug may actually be that we do something with it on 'select'. Inline replaced elements should honor 'height' but not 'line-height', just as it's the reverse for inline non-replaced elements (like links). cc:ing dbaron, because I just loooove to fill up his inbox. Also because he probably knows a lot more about forms and CSS than I do, particularly whatever work is being done for CSS3, and I'm thinking this bug should probably be moved over to Style anyway.
It shouldn't be moved over to style. I've always said that CSS shouldn't affect form controls, so I don't think the 'line-height' should have an affect here, but many others disagree with me. What's clearly happening is that it's being inherited into the inside of the form control, which we build out of basic CSS display types (blocks and inlines) and a bunch of fancier stuff.
So, it's better to disable or to finish the way we started? I am wondering what would be fast, safe and easier. We do have sites using this (this case came from a major site in china). Note that once you set height to the *input* the line-height have effect! also.
We'll be disabling line-height on inputs, looks like.
Assignee: rods → jkeiser
Fixed by checkin for bug 82265.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.