stylelint-plugin-defensive-css 0.6.0 → 0.8.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "stylelint-plugin-defensive-css",
-  "version": "0.6.0",
+  "version": "0.8.0",
   "description": "A Stylelint plugin to enforce defensive CSS best practices.",
   "main": "src/index.js",
   "files": [

package/src/rules/use-defensive-css/index.js

@@ -33,13 +33,12 @@ let isLastStyleDeclaration = false;
 let isWrappedInHoverAtRule = false;
 
 function traverseParentRules(parent) {
-  console.log({ parent });
   if (parent.parent.type === 'root') {
     return;
   }
 
   if (parent.parent.type === 'atrule') {
-    if (parent.parent.params.includes('hover: hover')) {
+    if (parent.parent.params && parent.parent.params.includes('hover: hover')) {
       isWrappedInHoverAtRule = true;
     } else {
       traverseParentRules(parent.parent);
@@ -64,7 +63,7 @@ const ruleFunction = (_, options) => {
       if (options?.['accidental-hover']) {
         const parent = decl.parent;
         const selector = parent.selector;
-        const isHoverSelector = selector.includes(':hover');
+        const isHoverSelector = selector?.includes(':hover');
         isWrappedInHoverAtRule = false;
 
         if (isHoverSelector) {
@@ -155,11 +154,19 @@ const ruleFunction = (_, options) => {
           flexWrappingProps.nodeToReport = decl;
         }
 
+        if (decl.prop === 'flex-flow' && decl.value.includes('column')) {
+          flexWrappingProps.isFlexRow = false;
+          flexWrappingProps.isMissingFlexWrap = false;
+        }
+
         if (decl.prop === 'flex-direction' && decl.value.includes('column')) {
           flexWrappingProps.isFlexRow = false;
         }
 
-        if (decl.prop === 'flex-wrap') {
+        if (
+          decl.prop === 'flex-wrap' ||
+          (decl.prop === 'flex-flow' && decl.value.includes('wrap'))
+        ) {
           flexWrappingProps.isMissingFlexWrap = false;
         }