Jforrester has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/386726 )
Change subject: build: Upgrade stylelint and related devDependencies to latest
......................................................................
build: Upgrade stylelint and related devDependencies to latest
Change-Id: I4c47f1d290802705d061167fe369c130405512e0
---
M .stylelintrc.json
M package.json
M resources/src/mediawiki.action/mediawiki.action.history.css
M resources/src/mediawiki.legacy/commonPrint.css
M resources/src/mediawiki.legacy/shared.css
M resources/src/mediawiki.less/mediawiki.ui/mixins.less
M resources/src/mediawiki.rcfilters/styles/mw.rcfilters.mixins.less
M
resources/src/mediawiki.rcfilters/styles/mw.rcfilters.ui.ChangesListWrapperWidget.highlightCircles.seenunseen.less
M resources/src/mediawiki.rcfilters/styles/mw.rcfilters.ui.TagItemWidget.less
M resources/src/mediawiki.skinning/content.externallinks.css
M resources/src/mediawiki.skinning/content.parsoid.less
M resources/src/mediawiki.skinning/elements.css
M resources/src/mediawiki.special/mediawiki.special.comparepages.styles.less
M resources/src/mediawiki.ui/components/checkbox.less
M resources/src/mediawiki.ui/components/forms.less
M resources/src/mediawiki.ui/components/inputs.less
M resources/src/mediawiki.ui/components/radio.less
17 files changed, 79 insertions(+), 80 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/26/386726/1
diff --git a/.stylelintrc.json b/.stylelintrc.json
index 27e289d..a6b858c 100644
--- a/.stylelintrc.json
+++ b/.stylelintrc.json
@@ -2,7 +2,6 @@
"extends": "stylelint-config-wikimedia",
"rules": {
"no-descending-specificity": null,
-
- "selector-no-id": null
+ "selector-max-id": null
}
}
diff --git a/package.json b/package.json
index 58f48e4..faca87d 100644
--- a/package.json
+++ b/package.json
@@ -17,7 +17,7 @@
"grunt-eslint": "20.1.0",
"grunt-jsonlint": "1.1.0",
"grunt-karma": "2.0.0",
- "grunt-stylelint": "0.8.0",
+ "grunt-stylelint": "0.9.0",
"grunt-webdriver": "2.0.3",
"karma": "1.7.1",
"karma-chrome-launcher": "2.2.0",
@@ -26,8 +26,8 @@
"karma-qunit": "1.2.1",
"nodemw": "0.11.0",
"qunitjs": "2.4.1",
- "stylelint": "7.8.0",
- "stylelint-config-wikimedia": "0.4.1",
+ "stylelint": "8.2.0",
+ "stylelint-config-wikimedia": "0.4.2",
"wdio-junit-reporter": "0.2.0",
"wdio-mocha-framework": "0.5.8",
"wdio-sauce-service": "^0.3.1",
diff --git a/resources/src/mediawiki.action/mediawiki.action.history.css
b/resources/src/mediawiki.action/mediawiki.action.history.css
index f3ea163..3c95be9 100644
--- a/resources/src/mediawiki.action/mediawiki.action.history.css
+++ b/resources/src/mediawiki.action/mediawiki.action.history.css
@@ -1,6 +1,6 @@
/* Styles for the JavaScript enhancements of the history page */
#pagehistory li.before input[name='oldid'],
-#pagehistory li.after input[name='diff'] {
+ #pagehistory li.after input[name='diff'] {
visibility: hidden;
}
diff --git a/resources/src/mediawiki.legacy/commonPrint.css
b/resources/src/mediawiki.legacy/commonPrint.css
index 8ff0634..4dd794b 100644
--- a/resources/src/mediawiki.legacy/commonPrint.css
+++ b/resources/src/mediawiki.legacy/commonPrint.css
@@ -78,7 +78,7 @@
/* Expand protocol-relative URLs for printing */
.mw-body-content a.external.text[href^='//']:after,
-.mw-body-content a.external.autonumber[href^='//']:after {
+ .mw-body-content a.external.autonumber[href^='//']:after {
content: ' (https:' attr( href ) ')';
}
diff --git a/resources/src/mediawiki.legacy/shared.css
b/resources/src/mediawiki.legacy/shared.css
index 74233d6..ecdfb61 100644
--- a/resources/src/mediawiki.legacy/shared.css
+++ b/resources/src/mediawiki.legacy/shared.css
@@ -56,36 +56,36 @@
/* Input types that should follow user direction, like buttons */
/* TODO: What about buttons in wikipage content ? */
-input[type='submit'],
-input[type='button'],
-input[type='reset'],
-input[type='file'] {
+input[ type='submit' ],
+input[ type='button' ],
+input[ type='reset' ],
+input[ type='file' ] {
direction: ltr;
}
/* Override default values */
-textarea[dir='ltr'],
-input[dir='ltr'] {
+textarea[ dir='ltr' ],
+input[ dir='ltr' ] {
/* @noflip */
direction: ltr;
}
-textarea[dir='rtl'],
-input[dir='rtl'] {
+textarea[ dir='rtl' ],
+input[ dir='rtl' ] {
/* @noflip */
direction: rtl;
}
/* Default style for semantic tags */
-abbr[title],
-.explain[title] {
+abbr[ title ],
+.explain[ title ] {
border-bottom: 1px dotted;
cursor: help;
}
@supports ( text-decoration: underline dotted ) {
- abbr[title],
- .explain[title] {
+ abbr[ title ],
+ .explain[ title ] {
border-bottom: 0;
text-decoration: underline dotted;
}
diff --git a/resources/src/mediawiki.less/mediawiki.ui/mixins.less
b/resources/src/mediawiki.less/mediawiki.ui/mixins.less
index 505aa61..1e078bd 100644
--- a/resources/src/mediawiki.less/mediawiki.ui/mixins.less
+++ b/resources/src/mediawiki.less/mediawiki.ui/mixins.less
@@ -17,8 +17,8 @@
line-height: normal;
font-weight: normal;
- & > input[type='checkbox'],
- & > input[type='radio'] {
+ & > input[ type='checkbox' ],
+ & > input[ type='radio' ] {
width: auto;
height: auto;
margin: 0 0.1em 0 0;
diff --git a/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.mixins.less
b/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.mixins.less
index f3d81b7..790e015 100644
--- a/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.mixins.less
+++ b/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.mixins.less
@@ -55,9 +55,9 @@
// work if the rules are inside the above widget LESS scope
.highlight-changesListWrapperWidget( @bgcolor ) {
.mw-rcfilters-ui-changesListWrapperWidget li&,
- .mw-rcfilters-ui-changesListWrapperWidget & tr:first-child,
- .mw-rcfilters-ui-changesListWrapperWidget
tr&.mw-rcfilters-ui-changesListWrapperWidget-enhanced-toplevel:not(.mw-rcfilters-ui-changesListWrapperWidget-enhanced-grey)
td:not( :nth-child( -n+2 ) ),
- .mw-rcfilters-ui-changesListWrapperWidget
tr&.mw-rcfilters-ui-changesListWrapperWidget-enhanced-nested:not(.mw-rcfilters-ui-changesListWrapperWidget-enhanced-grey)
td:not( :nth-child( -n+4 ) ) {
+ .mw-rcfilters-ui-changesListWrapperWidget & tr:first-child,
+ .mw-rcfilters-ui-changesListWrapperWidget
tr&.mw-rcfilters-ui-changesListWrapperWidget-enhanced-toplevel:not(
.mw-rcfilters-ui-changesListWrapperWidget-enhanced-grey ) td:not( :nth-child(
-n+2 ) ),
+ .mw-rcfilters-ui-changesListWrapperWidget
tr&.mw-rcfilters-ui-changesListWrapperWidget-enhanced-nested:not(
.mw-rcfilters-ui-changesListWrapperWidget-enhanced-grey ) td:not( :nth-child(
-n+4 ) ) {
background-color: @bgcolor;
}
}
diff --git
a/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.ui.ChangesListWrapperWidget.highlightCircles.seenunseen.less
b/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.ui.ChangesListWrapperWidget.highlightCircles.seenunseen.less
index deaee28..5f6fbea 100644
---
a/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.ui.ChangesListWrapperWidget.highlightCircles.seenunseen.less
+++
b/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.ui.ChangesListWrapperWidget.highlightCircles.seenunseen.less
@@ -21,7 +21,7 @@
.mw-rcfilters-mixin-circle(
@highlight-none, @result-circle-diameter, 0, true, @highlight-grey, true );
}
-
.mw-rcfilters-ui-changesListWrapperWidget:not(.mw-rcfilters-ui-changesListWrapperWidget-highlighted)
& {
+
.mw-rcfilters-ui-changesListWrapperWidget:not(
.mw-rcfilters-ui-changesListWrapperWidget-highlighted ) & {
.mw-rcfilters-mixin-circle(
@highlight-none, @result-circle-diameter, 0, true, @highlight-bluedot, true );
}
}
@@ -31,7 +31,7 @@
.mw-rcfilters-mixin-circle(
@highlight-grey, @result-circle-diameter, 0, true, @highlight-grey );
}
-
.mw-rcfilters-ui-changesListWrapperWidget:not(.mw-rcfilters-ui-changesListWrapperWidget-highlighted)
& {
+
.mw-rcfilters-ui-changesListWrapperWidget:not(
.mw-rcfilters-ui-changesListWrapperWidget-highlighted ) & {
.mw-rcfilters-mixin-circle(
@highlight-bluedot, @result-circle-diameter, 0, true, @highlight-bluedot );
}
}
diff --git
a/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.ui.TagItemWidget.less
b/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.ui.TagItemWidget.less
index 0e7a635..e9c982a 100644
---
a/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.ui.TagItemWidget.less
+++
b/resources/src/mediawiki.rcfilters/styles/mw.rcfilters.ui.TagItemWidget.less
@@ -58,23 +58,23 @@
top: 50%;
}
- &[data-color='c1']:before {
+ &[ data-color='c1' ]:before {
.mw-rcfilters-mixin-circle( @highlight-c1, 10px, ~'-5px
0.5em 0 0' );
}
- &[data-color='c2']:before {
+ &[ data-color='c2' ]:before {
.mw-rcfilters-mixin-circle( @highlight-c2, 10px, ~'-5px
0.5em 0 0' );
}
- &[data-color='c3']:before {
+ &[ data-color='c3' ]:before {
.mw-rcfilters-mixin-circle( @highlight-c3, 10px, ~'-5px
0.5em 0 0' );
}
- &[data-color='c4']:before {
+ &[ data-color='c4' ]:before {
.mw-rcfilters-mixin-circle( @highlight-c4, 10px, ~'-5px
0.5em 0 0' );
}
- &[data-color='c5']:before {
+ &[ data-color='c5' ]:before {
.mw-rcfilters-mixin-circle( @highlight-c5, 10px, ~'-5px
0.5em 0 0' );
}
}
diff --git a/resources/src/mediawiki.skinning/content.externallinks.css
b/resources/src/mediawiki.skinning/content.externallinks.css
index cd674ef..2e3aef1 100644
--- a/resources/src/mediawiki.skinning/content.externallinks.css
+++ b/resources/src/mediawiki.skinning/content.externallinks.css
@@ -20,7 +20,7 @@
}
.mw-body-content a.external[href^='mailto:'],
-.link-mailto {
+ .link-mailto {
background: url( images/mail.png ) center right no-repeat;
/* @embed */
background-image: linear-gradient( transparent, transparent ), url(
images/mail.svg );
@@ -28,7 +28,7 @@
}
.mw-body-content a.external[href^='ftp://'],
-.link-ftp {
+ .link-ftp {
background: url( images/ftp-ltr.png ) center right no-repeat;
/* @embed */
background-image: linear-gradient( transparent, transparent ), url(
images/ftp-ltr.svg );
@@ -36,8 +36,8 @@
}
.mw-body-content a.external[href^='irc://'],
-.mw-body-content a.external[href^='ircs://'],
-.link-irc {
+ .mw-body-content a.external[href^='ircs://'],
+ .link-irc {
background: url( images/chat-ltr.png ) center right no-repeat;
/* @embed */
background-image: linear-gradient( transparent, transparent ), url(
images/chat-ltr.svg );
@@ -45,18 +45,18 @@
}
.mw-body-content a.external[href$='.ogg'],
-.mw-body-content a.external[href$='.OGG'],
-.mw-body-content a.external[href$='.mid'],
-.mw-body-content a.external[href$='.MID'],
-.mw-body-content a.external[href$='.midi'],
-.mw-body-content a.external[href$='.MIDI'],
-.mw-body-content a.external[href$='.mp3'],
-.mw-body-content a.external[href$='.MP3'],
-.mw-body-content a.external[href$='.wav'],
-.mw-body-content a.external[href$='.WAV'],
-.mw-body-content a.external[href$='.wma'],
-.mw-body-content a.external[href$='.WMA'],
-.link-audio {
+ .mw-body-content a.external[href$='.OGG'],
+ .mw-body-content a.external[href$='.mid'],
+ .mw-body-content a.external[href$='.MID'],
+ .mw-body-content a.external[href$='.midi'],
+ .mw-body-content a.external[href$='.MIDI'],
+ .mw-body-content a.external[href$='.mp3'],
+ .mw-body-content a.external[href$='.MP3'],
+ .mw-body-content a.external[href$='.wav'],
+ .mw-body-content a.external[href$='.WAV'],
+ .mw-body-content a.external[href$='.wma'],
+ .mw-body-content a.external[href$='.WMA'],
+ .link-audio {
background: url( images/audio-ltr.png ) center right no-repeat;
/* @embed */
background-image: linear-gradient( transparent, transparent ), url(
images/audio-ltr.svg );
@@ -64,14 +64,14 @@
}
.mw-body-content a.external[href$='.ogm'],
-.mw-body-content a.external[href$='.OGM'],
-.mw-body-content a.external[href$='.avi'],
-.mw-body-content a.external[href$='.AVI'],
-.mw-body-content a.external[href$='.mpeg'],
-.mw-body-content a.external[href$='.MPEG'],
-.mw-body-content a.external[href$='.mpg'],
-.mw-body-content a.external[href$='.MPG'],
-.link-video {
+ .mw-body-content a.external[href$='.OGM'],
+ .mw-body-content a.external[href$='.avi'],
+ .mw-body-content a.external[href$='.AVI'],
+ .mw-body-content a.external[href$='.mpeg'],
+ .mw-body-content a.external[href$='.MPEG'],
+ .mw-body-content a.external[href$='.mpg'],
+ .mw-body-content a.external[href$='.MPG'],
+ .link-video {
background: url( images/video.png ) center right no-repeat;
/* @embed */
background-image: linear-gradient( transparent, transparent ), url(
images/video.svg );
@@ -79,12 +79,12 @@
}
.mw-body-content a.external[href$='.pdf'],
-.mw-body-content a.external[href$='.PDF'],
-.mw-body-content a.external[href*='.pdf#'],
-.mw-body-content a.external[href*='.PDF#'],
-.mw-body-content a.external[href*='.pdf?'],
-.mw-body-content a.external[href*='.PDF?'],
-.link-document {
+ .mw-body-content a.external[href$='.PDF'],
+ .mw-body-content a.external[href*='.pdf#'],
+ .mw-body-content a.external[href*='.PDF#'],
+ .mw-body-content a.external[href*='.pdf?'],
+ .mw-body-content a.external[href*='.PDF?'],
+ .link-document {
background: url( images/document-ltr.png ) center right no-repeat;
/* @embed */
background-image: linear-gradient( transparent, transparent ), url(
images/document-ltr.svg );
diff --git a/resources/src/mediawiki.skinning/content.parsoid.less
b/resources/src/mediawiki.skinning/content.parsoid.less
index 654b655..a36ba6c 100644
--- a/resources/src/mediawiki.skinning/content.parsoid.less
+++ b/resources/src/mediawiki.skinning/content.parsoid.less
@@ -44,9 +44,9 @@
/**
* Block media items
*/
-figure[typeof*='mw:Image'],
-figure[typeof*='mw:Video'],
-figure[typeof*='mw:Audio'] {
+figure[ typeof*='mw:Image' ],
+figure[ typeof*='mw:Video' ],
+figure[ typeof*='mw:Audio' ] {
margin: 0;
a {
@@ -91,12 +91,12 @@
}
}
-figure[typeof~='mw:Image/Thumb'],
-figure[typeof~='mw:Video/Thumb'],
-figure[typeof~='mw:Audio/Thumb'],
-figure[typeof~='mw:Image/Frame'],
-figure[typeof~='mw:Video/Frame'],
-figure[typeof~='mw:Audio/Frame'] {
+figure[ typeof~='mw:Image/Thumb' ],
+figure[ typeof~='mw:Video/Thumb' ],
+figure[ typeof~='mw:Audio/Thumb' ],
+figure[ typeof~='mw:Image/Frame' ],
+figure[ typeof~='mw:Video/Frame' ],
+figure[ typeof~='mw:Audio/Frame' ] {
display: table;
text-align: center;
border: 1px solid #c8ccd1;
@@ -139,9 +139,9 @@
}
}
-figure[typeof*='mw:Image/Thumb'],
-figure[typeof*='mw:Video/Thumb'],
-figure[typeof*='mw:Audio/Thumb'] {
+figure[ typeof*='mw:Image/Thumb' ],
+figure[ typeof*='mw:Video/Thumb' ],
+figure[ typeof*='mw:Audio/Thumb' ] {
> a:after {
content: '';
width: 15px;
diff --git a/resources/src/mediawiki.skinning/elements.css
b/resources/src/mediawiki.skinning/elements.css
index 245fb53..366c5a9 100644
--- a/resources/src/mediawiki.skinning/elements.css
+++ b/resources/src/mediawiki.skinning/elements.css
@@ -13,7 +13,7 @@
background: none;
}
-a:not( [href] ) {
+a:not( [ href ] ) {
cursor: pointer; /* Always cursor:pointer even without href */
}
diff --git
a/resources/src/mediawiki.special/mediawiki.special.comparepages.styles.less
b/resources/src/mediawiki.special/mediawiki.special.comparepages.styles.less
index c951169..87b7a8b 100644
--- a/resources/src/mediawiki.special/mediawiki.special.comparepages.styles.less
+++ b/resources/src/mediawiki.special/mediawiki.special.comparepages.styles.less
@@ -10,7 +10,7 @@
.box-sizing( border-box );
}
-.mw-special-ComparePages
.oo-ui-layout.oo-ui-panelLayout.oo-ui-panelLayout-padded.oo-ui-panelLayout-framed:nth-of-type(2)
{
+.mw-special-ComparePages
.oo-ui-layout.oo-ui-panelLayout.oo-ui-panelLayout-padded.oo-ui-panelLayout-framed:nth-of-type(
2 ) {
margin-left: 2%;
}
diff --git a/resources/src/mediawiki.ui/components/checkbox.less
b/resources/src/mediawiki.ui/components/checkbox.less
index c1626db..0c13daf 100644
--- a/resources/src/mediawiki.ui/components/checkbox.less
+++ b/resources/src/mediawiki.ui/components/checkbox.less
@@ -49,7 +49,7 @@
vertical-align: middle;
}
- [type='checkbox'] {
+ [ type='checkbox' ] {
display: table-cell;
position: relative;
// Ensure the invisible input takes up the required `width` &
`height`
diff --git a/resources/src/mediawiki.ui/components/forms.less
b/resources/src/mediawiki.ui/components/forms.less
index d3c76d0..0a9023e 100644
--- a/resources/src/mediawiki.ui/components/forms.less
+++ b/resources/src/mediawiki.ui/components/forms.less
@@ -66,7 +66,7 @@
}
// Override input styling just for checkboxes and radio inputs.
- input[type='radio'] {
+ input[ type='radio' ] {
display: inline;
.box-sizing( content-box );
width: auto;
diff --git a/resources/src/mediawiki.ui/components/inputs.less
b/resources/src/mediawiki.ui/components/inputs.less
index 74dc0b7..1c79d52 100644
--- a/resources/src/mediawiki.ui/components/inputs.less
+++ b/resources/src/mediawiki.ui/components/inputs.less
@@ -73,7 +73,7 @@
}
// Normalize styling for `<input type="search">`
- &[type='search'] {
+ &[ type='search' ] {
// Correct the odd appearance in Chrome and Safari 5
-webkit-appearance: textfield;
@@ -103,7 +103,7 @@
// <button class="mw-ui-button mw-ui-progressive">Submit</button>
//
// Styleguide 1.2.
-input[type='number'],
+input[ type='number' ],
.mw-ui-input-inline {
display: inline-block;
width: auto;
diff --git a/resources/src/mediawiki.ui/components/radio.less
b/resources/src/mediawiki.ui/components/radio.less
index 3d82e8e..61c6ceb 100644
--- a/resources/src/mediawiki.ui/components/radio.less
+++ b/resources/src/mediawiki.ui/components/radio.less
@@ -48,7 +48,7 @@
vertical-align: middle;
}
- [type='radio'] {
+ [ type='radio' ] {
// ensure the invisible radio takes up the required width
width: @sizeInputBinary;
height: @sizeInputBinary;
--
To view, visit https://gerrit.wikimedia.org/r/386726
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4c47f1d290802705d061167fe369c130405512e0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Jforrester <jforres...@wikimedia.org>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
