[wp-trac] [WordPress Trac] #28500: css issue FF30 input height

WordPress Trac noreply at wordpress.org
Tue Jun 17 19:27:06 UTC 2014


#28500: css issue FF30 input height
----------------------------+---------------------------------
 Reporter:  alexufo         |       Owner:
     Type:  defect (bug)    |      Status:  new
 Priority:  normal          |   Milestone:  4.0
Component:  Administration  |     Version:  trunk
 Severity:  normal          |  Resolution:
 Keywords:  needs-patch     |     Focuses:  ui, administration
----------------------------+---------------------------------

Comment (by azaozz):

 Replying to [comment:7 afercia]:

 Thanks for the detailed explanation :)

 > For years we had to deal with Firefox default declaration line-height:
 normal !important and every line-height value in our stylesheets didn't
 apply to Firefox. So all our rules on input fields were crafted to match
 Firefox behavior.
 > ...
 > Ironically, the quickest fix would be to re-introduce what Mozilla
 removed and just add
 > {{{
 > input {
 >   line-height: normal !important;
 > }
 > }}}
 > because we can assume all the other line-height declarations for input
 fields that set a specific value, are there to match Firefox previous
 behavior that was, that's the point: normal!
 >
 > On the other hand, that would be a missed opportunity to have better CSS
 but the only thing that needs to be adjusted is the line-height on inputs,
 no need to adjust "height" or other properties.

 Yes, by the time WordPress 4.0 is released (couple of months), the great
 majority of Firefox users will be on 30+ so thinking it would be best to
 take advantage of this change and improve our CSS.

 One thing to be aware of is text input fields with larger font size that
 also have more padding. Sometimes the top and bottom of the text can be
 cut off.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/28500#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list