MDL-55452 stylelint: prevent use of !important
authorDan Poltawski <dan@moodle.com>
Fri, 12 Aug 2016 11:02:23 +0000 (12:02 +0100)
committerDan Poltawski <dan@moodle.com>
Fri, 26 Aug 2016 10:08:14 +0000 (11:08 +0100)
Don't try and fix existing issues, just prevent new ones landing..

.stylelintrc
theme/bootstrapbase/less/moodle/course.less
theme/bootstrapbase/less/moodle/expendable.less
theme/bootstrapbase/less/moodle/filemanager.less
theme/bootstrapbase/less/moodle/modules.less
theme/bootstrapbase/less/moodle/responsive.less
theme/bootstrapbase/less/moodle/undo.less

index e5e61a2..572cd2f 100644 (file)
@@ -31,6 +31,7 @@
         "declaration-colon-newline-after": "always-multi-line",
         "declaration-colon-space-after": "always-single-line",
         "declaration-colon-space-before": "never",
+        "declaration-no-important": true,
         "function-calc-no-unspaced-operator": true,
         "function-comma-newline-after": "always-multi-line",
         "function-comma-space-after": "always-single-line",
index 571e3d8..a64b41b 100644 (file)
@@ -850,7 +850,8 @@ span.editinstructions {
     }
     .moodle-actionmenu {
         .iconsmall {
-            max-width: none !important; /** reset sets 100% !important which breaks on IE8 without this !important */
+            /** reset sets 100% !important which breaks on IE8 without this !important */
+            max-width: none !important; /* stylelint-disable-line declaration-no-important */
             width: 16px;
             height: 16px;
             padding: 4px;
index f476212..3d25419 100644 (file)
@@ -1,3 +1,4 @@
+/* stylelint-disable declaration-no-important */
 table#explaincaps,
 table#defineroletable,
 table.grading-report,
index 468207f..13b8309 100644 (file)
@@ -1,3 +1,4 @@
+/* stylelint-disable declaration-no-important */
 // File Picker and File Manager
 .filemanager,
 .filepicker,
index faba707..29b0eb4 100644 (file)
@@ -136,7 +136,7 @@ select {
 
 // Override hardcoded forum modules styling
 .forumsearch input[type=text] {
-    margin-bottom: 0 !important;
+    margin-bottom: 0 !important; /* stylelint-disable-line declaration-no-important */
 }
 
 #page-mod-forum-discuss .discussioncontrols {
index 5f8fb00..2ad216e 100644 (file)
         padding-right: 10px;
     }
     #helppopupbox {
+        /* stylelint-disable declaration-no-important */
         width: auto !important;
         left: 0 !important;
+        /* stylelint-enable declaration-no-important */
     }
 }
 // Shrink calender when short on space in block
index 18726b1..effa31a 100644 (file)
@@ -69,10 +69,9 @@ li.section.hidden,
  * basically if you see things shifted left or right compared
  * with where they should be check for a .row
  */
-
 #turnitintool_style .row,
 .forumpost .row {
-    margin-left: 0 !important;
+    margin-left: 0 !important; /* stylelint-disable-line declaration-no-important */
     // not sure if this needs !important
 }
 #turnitintool_style .row:before,