Bug 1689477 - Remove number-input padding special-case so that we size them the same way as text controls. r=spohl
authorEmilio Cobos Álvarez <emilio@crisal.io>
Thu, 28 Jan 2021 23:22:51 +0000
changeset 565345 d94527834d1ddc2f75173ed650c8936380701853
parent 565344 8f869fb94e1b0dec045dcee36a7f3153bdbb6df6
child 565346 7ce9fbfc137a43a737c159dd174fc0e67ee3ba18
push id38159
push usersmolnar@mozilla.com
push dateMon, 01 Feb 2021 09:44:43 +0000
treeherdermozilla-central@7cdfc29ee187 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersspohl
bugs1689477
milestone87.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1689477 - Remove number-input padding special-case so that we size them the same way as text controls. r=spohl This means that the number input by default shows some white space to the right of the spinners. I think it's probably not the end of the world, and depending on the different trade-offs we might want to do this instead of fixing the test. Depends on D103270 Differential Revision: https://phabricator.services.mozilla.com/D103271
layout/style/res/forms.css
--- a/layout/style/res/forms.css
+++ b/layout/style/res/forms.css
@@ -992,22 +992,16 @@ input[type=number]::-moz-number-spin-dow
   border-bottom-right-radius: 4px;
 }
 
 input[type=number] > div > div > div:hover {
   /* Give some indication of hover state for the up/down buttons */
   background-color: lightblue;
 }
 
-@media (-moz-non-native-content-theme) {
-  input[type=number] {
-    padding-inline-end: 0;
-  }
-}
-
 input[type=search]::-moz-search-clear-button {
   display: block;
   cursor: default;
   width: 1em;
   height: 1em;
   max-height: 1em;
   margin-inline-start: 1px;
   align-self: center;