fix: escape current selector for focusable (#122)

warp-ds · Jun 14, 2023 · 00b789b · 00b789b
1 parent 3397132
commit 00b789b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/src/_rules/focus-ring.js b/src/_rules/focus-ring.js
@@ -34,8 +34,9 @@ export const focusRing = [
       }
       return `.${escapedSelector}:${selectorWithVariant.split(':')?.[0]}{${focusRingStyle}}`;
     } else {
-      const focus = `.${currentSelector}:focus,.${currentSelector}:focus-visible{${focusRingStyle}}`;
-      const notFocusVisible = `.${currentSelector}:not(:focus-visible){${outlineNone}}`;
+      const escapedCurrentSelector = escapeSelector(currentSelector);
+      const focus = `.${escapedCurrentSelector}:focus,.${escapedCurrentSelector}:focus-visible{${focusRingStyle}}`;
+      const notFocusVisible = `.${escapedCurrentSelector}:not(:focus-visible){${outlineNone}}`;
       return focus + notFocusVisible;
     }
   }],