From b4881cc18a88d6a891a7d7edc58953d413ee1ab5 Mon Sep 17 00:00:00 2001
From: Jelle van der Waa <jvanderwaa@redhat.com>
Date: Mon, 4 Nov 2024 20:01:19 +0100
Subject: [PATCH] stylelintrc: sync stylelint config from Cockpit

---
 .stylelintrc.json | 63 ++++++++++++++++++++++++-----------------------
 src/Images.css    |  2 +-
 2 files changed, 33 insertions(+), 32 deletions(-)

diff --git a/.stylelintrc.json b/.stylelintrc.json
index 67a29f062..762bf194c 100644
--- a/.stylelintrc.json
+++ b/.stylelintrc.json
@@ -1,39 +1,40 @@
 {
   "extends": "stylelint-config-standard-scss",
   "plugins": [
-	"stylelint-use-logical-spec"
+    "stylelint-use-logical-spec"
   ],
   "rules": {
-	"at-rule-empty-line-before": null,
-	"declaration-empty-line-before": null,
-	"custom-property-empty-line-before": null,
-	"comment-empty-line-before": null,
-	"scss/double-slash-comment-empty-line-before": null,
-	"scss/dollar-variable-colon-space-after": null,
+    "at-rule-empty-line-before": null,
+    "declaration-empty-line-before": null,
+    "custom-property-empty-line-before": null,
+    "comment-empty-line-before": null,
+    "scss/double-slash-comment-empty-line-before": null,
+    "scss/dollar-variable-colon-space-after": null,
 
-	"custom-property-pattern": null,
-	"declaration-block-no-duplicate-properties": null,
-	"declaration-block-no-redundant-longhand-properties": null,
-	"declaration-block-no-shorthand-property-overrides": null,
-	"declaration-block-single-line-max-declarations": null,
-	"font-family-no-duplicate-names": null,
-	"function-url-quotes": null,
-	"keyframes-name-pattern": null,
-	"media-feature-range-notation": "prefix",
-	"no-descending-specificity": null,
-	"no-duplicate-selectors": null,
-	"scss/at-extend-no-missing-placeholder": null,
-	"scss/at-import-partial-extension": null,
-	"scss/at-import-no-partial-leading-underscore": null,
-	"scss/load-no-partial-leading-underscore": true,
-	"scss/at-mixin-pattern": null,
-	"scss/comment-no-empty": null,
-	"scss/dollar-variable-pattern": null,
-	"scss/double-slash-comment-whitespace-inside": null,
-	"scss/no-global-function-names": null,
-	"scss/operator-no-unspaced": null,
-	"selector-class-pattern": null,
-	"selector-id-pattern": null,
-	"liberty/use-logical-spec": "always"
+    "custom-property-pattern": null,
+    "declaration-block-no-duplicate-properties": null,
+    "declaration-block-no-redundant-longhand-properties": null,
+    "declaration-block-no-shorthand-property-overrides": null,
+    "declaration-block-single-line-max-declarations": null,
+    "font-family-no-duplicate-names": null,
+    "function-url-quotes": null,
+    "keyframes-name-pattern": null,
+    "media-feature-range-notation": "prefix",
+    "no-descending-specificity": null,
+    "no-duplicate-selectors": null,
+    "scss/at-extend-no-missing-placeholder": null,
+    "scss/at-import-partial-extension": null,
+    "scss/at-import-no-partial-leading-underscore": null,
+    "scss/load-no-partial-leading-underscore": null,
+    "scss/at-mixin-pattern": null,
+    "scss/comment-no-empty": null,
+    "scss/dollar-variable-pattern": null,
+    "scss/double-slash-comment-whitespace-inside": null,
+    "scss/no-global-function-names": null,
+    "scss/operator-no-unspaced": null,
+    "selector-class-pattern": null,
+    "selector-id-pattern": null,
+    "selector-not-notation": "simple",
+    "liberty/use-logical-spec": "always"
   }
 }
diff --git a/src/Images.css b/src/Images.css
index b93b2acaa..347f48306 100644
--- a/src/Images.css
+++ b/src/Images.css
@@ -4,7 +4,7 @@
 }
 
 /* Danger dropdown items should be red */
-.pf-v5-c-dropdown__menu-item.pf-m-danger:not(.pf-m-disabled, .pf-m-aria-disabled) {
+.pf-v5-c-dropdown__menu-item.pf-m-danger:not(.pf-m-disabled):not(.pf-m-aria-disabled) {
     color: var(--pf-v5-global--danger-color--200);
 }