build: Organise Gruntfile.js more consistently

This also adds linting for skinStyles, with a few fixes to make these pass.

Change-Id: Id26b0bad21850d985eaaceda24fa50ad6f886458

Gruntfile.js

@@ -13,10 +13,10 @@ module.exports = function ( grunt ) {
 		eslint: {
 			all: [
 				'**/*.js',
-				'!libs/**',
-				'!vendor/**',
 				'!docs/**',
-				'!node_modules/**'
+				'!libs/**',
+				'!node_modules/**',
+				'!vendor/**'
 			]
 		},
 		stylelint: {
@@ -24,17 +24,30 @@ module.exports = function ( grunt ) {
 				syntax: 'less'
 			},
 			all: [
-				'minerva.less/**/*.less',
-				'resources/**/*.less'
+				'**/*.less',
+				'!docs/**',
+				'!libs/**',
+				'!node_modules/**',
+				'!vendor/**'
 			]
 		},
+		jsonlint: {
+			all: [
+				'**/*.json',
+				'!docs/**',
+				'!libs/**',
+				'!node_modules/**',
+				'!vendor/**'
+			]
+		},
+		banana: conf.MessagesDirs,
 		watch: {
 			lint: {
-				files: [ 'resources/**/*.js', 'tests/qunit/**/*.js' ],
+				files: [ '{resources,tests/qunit}/**/*.{js,less}' ],
 				tasks: [ 'lint' ]
 			},
 			scripts: {
-				files: [ 'resources/**/*.js', 'tests/qunit/**/*.js' ],
+				files: [ '{resources,tests/qunit}/**/*.js' ],
 				tasks: [ 'test' ]
 			},
 			configFiles: {
@@ -43,19 +56,10 @@ module.exports = function ( grunt ) {
 					reload: true
 				}
 			}
-		},
-		banana: conf.MessagesDirs,
-		jsonlint: {
-			all: [
-				'*.json',
-				'**/*.json',
-				'!node_modules/**',
-				'!vendor/**'
-			]
 		}
 	} );
 
-	grunt.registerTask( 'lint', [ 'eslint', 'jsonlint', 'banana', 'stylelint' ] );
+	grunt.registerTask( 'lint', [ 'eslint', 'stylelint', 'jsonlint', 'banana' ] );
 	grunt.registerTask( 'test', [ 'lint' ] );
 
 	grunt.registerTask( 'default', [ 'test' ] );

skinStyles/mediawiki.hlist/minerva.less

@@ -1,4 +1,4 @@
- @import 'minerva.variables.less';
+@import 'minerva.variables.less';
 
 /**
  * This tweaks the default mediawiki.hlist module to provide performance optimisations
@@ -7,7 +7,7 @@
 
 // FIXME: to support hlists on enwiki projects - this should be deprecated
 .hlist > ul,
-.hlist > dl,.hlist > ul,
+.hlist > dl,
 // Horizontal Lists
 //
 // Use hlist class when dealing with lists where items should be horizontal.

skinStyles/mediawiki.special.preferences.styles/styles.less

@@ -46,4 +46,4 @@
 		border: solid 1px @colorFieldBorder;
 		padding: 8px;
 	}
-}
+}

skinStyles/mobile.startup/Overlay.less

@@ -4,6 +4,10 @@
 .overlay {
 	padding-top: @headerHeight;
 
+	> ul, button {
+		width: @headerHeight;
+	}
+
 	.license {
 		.secondary-text();
 	}
@@ -18,10 +22,6 @@
 	.slider-button {
 		top: @headerHeight;
 	}
-
-	> ul, button {
-		width: @headerHeight;
-	}
 }
 
 @media all and ( min-width: @width-breakpoint-desktop ) {