Minor Sass consistency changes. (#24677)

* use `background-color` instead of the shorthand
* use `outline: 0` consistently
* fix transform order
* remove quotes from `SFMono-Regular`

diff --git a/scss/_close.scss b/scss/_close.scss
index 897d4868443ab748ecd4cd22ce437ec2bda010a9..4271bc720b2ad985e53c2233c395bb5ea393c7ba 100644 (file)
--- a/scss/_close.scss
+++ b/scss/_close.scss
@@ -22,7 +22,7 @@
 // stylelint-disable property-no-vendor-prefix, selector-no-qualifying-type
 button.close {
   padding: 0;
-  background: transparent;
+  background-color: transparent;
   border: 0;
   -webkit-appearance: none;
 }

diff --git a/scss/_custom-forms.scss b/scss/_custom-forms.scss
index a521dbdc9445db37c507e036abb7c73b8b6055b6..643dab6ff6ef0cffc69706b165b46d7eed1e4d4c 100644 (file)
--- a/scss/_custom-forms.scss
+++ b/scss/_custom-forms.scss
@@ -148,7 +148,7 @@
 
   &:focus {
     border-color: $custom-select-focus-border-color;
-    outline: none;
+    outline: 0;
     @include box-shadow($custom-select-focus-box-shadow);
 
     &::-ms-value {

diff --git a/scss/_dropdown.scss b/scss/_dropdown.scss
index 19edfca7a7828b7f44ed28380a70eaafdb2f703e..a9d4cfed2e4abd01e52b584bb8182fabd3697f6d 100644 (file)
--- a/scss/_dropdown.scss
+++ b/scss/_dropdown.scss
@@ -89,7 +89,7 @@
   color: $dropdown-link-color;
   text-align: inherit; // For `<button>`s
   white-space: nowrap; // prevent links from randomly breaking onto new lines
-  background: none; // For `<button>`s
+  background-color: transparent; // For `<button>`s
   border: 0; // For `<button>`s
 
   @include hover-focus {

diff --git a/scss/_navbar.scss b/scss/_navbar.scss
index 6b023e82e4001979088b95a2f12366b0b80c3108..64766eb768a314384209604ac83ad18e9a2fc264 100644 (file)
--- a/scss/_navbar.scss
+++ b/scss/_navbar.scss
@@ -109,7 +109,7 @@
   padding: $navbar-toggler-padding-y $navbar-toggler-padding-x;
   font-size: $navbar-toggler-font-size;
   line-height: 1;
-  background: transparent; // remove default button style
+  background-color: transparent; // remove default button style
   border: $border-width solid transparent; // remove default button style
   @include border-radius($navbar-toggler-border-radius);
 

diff --git a/scss/_reboot.scss b/scss/_reboot.scss
index f98a7194785e664f2a266ae637fc654fafff9abb..62b3a666506a2702b7daa24824d6cdd1dca6d44d 100644 (file)
--- a/scss/_reboot.scss
+++ b/scss/_reboot.scss
@@ -71,7 +71,7 @@ body {
 //
 // Credit: https://github.com/suitcss/base
 [tabindex="-1"]:focus {
-  outline: none !important;
+  outline: 0 !important;
 }
 
 

diff --git a/scss/_variables.scss b/scss/_variables.scss
index 895dfc22f1b3b715951646d82a9038cbc32a42c2..a8e4392ab6ab4f8fff25f6a94e71a9fa52cbcc6e 100644 (file)
--- a/scss/_variables.scss
+++ b/scss/_variables.scss
@@ -221,7 +221,7 @@ $transition-collapse:         height .35s ease !default;
 
 // stylelint-disable value-keyword-case
 $font-family-sans-serif:      -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, "Helvetica Neue", Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol" !default;
-$font-family-monospace:       "SFMono-Regular", Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace !default;
+$font-family-monospace:       SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace !default;
 $font-family-base:            $font-family-sans-serif !default;
 // stylelint-enable value-keyword-case
 
@@ -416,7 +416,7 @@ $input-height-sm:                       calc(#{$input-height-inner-sm} + #{$inpu
 $input-height-inner-lg:                 ($font-size-lg * $input-btn-line-height-lg) + ($input-btn-padding-y-lg * 2) !default;
 $input-height-lg:                       calc(#{$input-height-inner-lg} + #{$input-height-border}) !default;
 
-$input-transition:                      border-color ease-in-out .15s, box-shadow ease-in-out .15s !default;
+$input-transition:                      border-color .15s ease-in-out, box-shadow .15s ease-in-out !default;
 
 $form-text-margin-top:                  .25rem !default;
 

diff --git a/scss/mixins/_forms.scss b/scss/mixins/_forms.scss
index 5d822a440d39b78fc0d072ded56e10a0a0727ffd..6c171aae8cf0f0882f2368bc2bcc0c44e34d1bf9 100644 (file)
--- a/scss/mixins/_forms.scss
+++ b/scss/mixins/_forms.scss
@@ -15,7 +15,7 @@
     color: $input-focus-color;
     background-color: $input-focus-bg;
     border-color: $input-focus-border-color;
-    outline: none;
+    outline: 0;
     // Avoid using mixin so we can pass custom focus shadow properly
     @if $enable-shadows {
       box-shadow: $input-box-shadow, $input-focus-box-shadow;