Cleanup stylelint comments

MartijnCuppens · XhmikosR · commit d0b7eb1111c7 · 2018-10-23T00:21:33.000+03:00
diff --git a/scss/_close.scss b/scss/_close.scss
@@ -25,11 +25,10 @@
 // If you want the anchor version, it requires `href="#"`.
 // See https://developer.mozilla.org/en-US/docs/Web/Events/click#Safari_Mobile
 
-// stylelint-disable property-no-vendor-prefix, selector-no-qualifying-type
+// stylelint-disable-next-line selector-no-qualifying-type
 button.close {
   padding: 0;
   background-color: transparent;
   border: 0;
-  -webkit-appearance: none;
+  appearance: none;
 }
-// stylelint-enable
diff --git a/scss/_dropdown.scss b/scss/_dropdown.scss
@@ -105,7 +105,7 @@
 }
 
 // When enabled Popper.js, reset basic dropdown position
-// stylelint-disable no-duplicate-selectors
+// stylelint-disable-next-line no-duplicate-selectors
 .dropdown-menu {
   &[x-placement^="top"],
   &[x-placement^="right"],
@@ -115,7 +115,6 @@
     bottom: auto;
   }
 }
-// stylelint-enable no-duplicate-selectors
 
 // Dividers (basically an `<hr>`) within the dropdown
 .dropdown-divider {
diff --git a/scss/_forms.scss b/scss/_forms.scss
@@ -155,18 +155,18 @@ select.form-control {
   @include border-radius($input-border-radius-lg);
 }
 
-// stylelint-disable no-duplicate-selectors
+// stylelint-disable-next-line no-duplicate-selectors
 select.form-control {
   &[size],
   &[multiple] {
     height: auto;
   }
 }
 
+// stylelint-disable-next-line no-duplicate-selectors
 textarea.form-control {
   height: auto;
 }
-// stylelint-enable no-duplicate-selectors
 
 // Form groups
 //
diff --git a/scss/_reboot.scss b/scss/_reboot.scss
@@ -40,12 +40,11 @@ html {
   }
 }
 
-// stylelint-disable selector-list-comma-newline-after
 // Shim for "new" HTML5 structural elements to display correctly (IE10, older browsers)
+// stylelint-disable-next-line selector-list-comma-newline-after
 article, aside, figcaption, figure, footer, header, hgroup, main, nav, section {
   display: block;
 }
-// stylelint-enable selector-list-comma-newline-after
 
 // Body
 //
@@ -95,12 +94,11 @@ hr {
 //
 // By default, `<h1>`-`<h6>` all receive top and bottom margins. We nuke the top
 // margin for easier control within type scales as it avoids margin collapsing.
-// stylelint-disable selector-list-comma-newline-after
+// stylelint-disable-next-line selector-list-comma-newline-after
 h1, h2, h3, h4, h5, h6 {
   margin-top: 0;
   margin-bottom: $headings-margin-bottom;
 }
-// stylelint-enable selector-list-comma-newline-after
 
 // Reset margins on paragraphs
 //
@@ -163,12 +161,10 @@ dfn {
   font-style: italic; // Add the correct font style in Android 4.3-
 }
 
-// stylelint-disable font-weight-notation
 b,
 strong {
   font-weight: bolder; // Add the correct font weight in Chrome, Edge, and Safari
 }
-// stylelint-enable font-weight-notation
 
 small {
   font-size: 80%; // Add the correct font size in all browsers
diff --git a/scss/mixins/_forms.scss b/scss/mixins/_forms.scss
@@ -80,7 +80,7 @@
     }
   }
 
-  // stylelint-disable selector-no-qualifying-type
+  // stylelint-disable-next-line selector-no-qualifying-type
   textarea.form-control {
     .was-validated &:#{$state},
     &.is-#{$state} {
@@ -90,7 +90,6 @@
       }
     }
   }
-  // stylelint-enable selector-no-qualifying-type
 
   .custom-select {
     .was-validated &:#{$state},

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`// When enabled Popper.js, reset basic dropdown position`
`108`		`-// stylelint-disable no-duplicate-selectors`
	`108`	`+// stylelint-disable-next-line no-duplicate-selectors`
`109`	`109`	`.dropdown-menu {`
`110`	`110`	`&[x-placement^="top"],`
`111`	`111`	`&[x-placement^="right"],`
`@@ -115,7 +115,6 @@`
`115`	`115`	`bottom: auto;`
`116`	`116`	`}`
`117`	`117`	`}`
`118`		`-// stylelint-enable no-duplicate-selectors`
`119`	`118`
`120`	`119`	// Dividers (basically an `<hr>`) within the dropdown
`121`	`120`	`.dropdown-divider {`
Original file line number	Diff line number	Diff line change
`@@ -155,18 +155,18 @@ select.form-control {`
`155`	`155`	`@include border-radius($input-border-radius-lg);`
`156`	`156`	`}`
`157`	`157`
`158`		`-// stylelint-disable no-duplicate-selectors`
	`158`	`+// stylelint-disable-next-line no-duplicate-selectors`
`159`	`159`	`select.form-control {`
`160`	`160`	`&[size],`
`161`	`161`	`&[multiple] {`
`162`	`162`	`height: auto;`
`163`	`163`	`}`
`164`	`164`	`}`
`165`	`165`
	`166`	`+// stylelint-disable-next-line no-duplicate-selectors`
`166`	`167`	`textarea.form-control {`
`167`	`168`	`height: auto;`
`168`	`169`	`}`
`169`		`-// stylelint-enable no-duplicate-selectors`
`170`	`170`
`171`	`171`	`// Form groups`
`172`	`172`	`//`
Original file line number	Diff line number	Diff line change
`@@ -40,12 +40,11 @@ html {`
`40`	`40`	`}`
`41`	`41`	`}`
`42`	`42`
`43`		`-// stylelint-disable selector-list-comma-newline-after`
`44`	`43`	`// Shim for "new" HTML5 structural elements to display correctly (IE10, older browsers)`
	`44`	`+// stylelint-disable-next-line selector-list-comma-newline-after`
`45`	`45`	`article, aside, figcaption, figure, footer, header, hgroup, main, nav, section {`
`46`	`46`	`display: block;`
`47`	`47`	`}`
`48`		`-// stylelint-enable selector-list-comma-newline-after`
`49`	`48`
`50`	`49`	`// Body`
`51`	`50`	`//`
`@@ -95,12 +94,11 @@ hr {`
`95`	`94`	`//`
`96`	`95`	// By default, `<h1>`-`<h6>` all receive top and bottom margins. We nuke the top
`97`	`96`	`// margin for easier control within type scales as it avoids margin collapsing.`
`98`		`-// stylelint-disable selector-list-comma-newline-after`
	`97`	`+// stylelint-disable-next-line selector-list-comma-newline-after`
`99`	`98`	`h1, h2, h3, h4, h5, h6 {`
`100`	`99`	`margin-top: 0;`
`101`	`100`	`margin-bottom: $headings-margin-bottom;`
`102`	`101`	`}`
`103`		`-// stylelint-enable selector-list-comma-newline-after`
`104`	`102`
`105`	`103`	`// Reset margins on paragraphs`
`106`	`104`	`//`
`@@ -163,12 +161,10 @@ dfn {`
`163`	`161`	`font-style: italic; // Add the correct font style in Android 4.3-`
`164`	`162`	`}`
`165`	`163`
`166`		`-// stylelint-disable font-weight-notation`
`167`	`164`	`b,`
`168`	`165`	`strong {`
`169`	`166`	`font-weight: bolder; // Add the correct font weight in Chrome, Edge, and Safari`
`170`	`167`	`}`
`171`		`-// stylelint-enable font-weight-notation`
`172`	`168`
`173`	`169`	`small {`
`174`	`170`	`font-size: 80%; // Add the correct font size in all browsers`
Original file line number	Diff line number	Diff line change
`@@ -80,7 +80,7 @@`
`80`	`80`	`}`
`81`	`81`	`}`
`82`	`82`
`83`		`- // stylelint-disable selector-no-qualifying-type`
	`83`	`+ // stylelint-disable-next-line selector-no-qualifying-type`
`84`	`84`	`textarea.form-control {`
`85`	`85`	`.was-validated &:#{$state},`
`86`	`86`	`&.is-#{$state} {`
`@@ -90,7 +90,6 @@`
`90`	`90`	`}`
`91`	`91`	`}`
`92`	`92`	`}`
`93`		`- // stylelint-enable selector-no-qualifying-type`
`94`	`93`
`95`	`94`	`.custom-select {`
`96`	`95`	`.was-validated &:#{$state},`