chore: remove focus-visible polyfill from local dev server (#420)

nolanlawson · Apr 13, 2024 · 1cd4b9d · 1cd4b9d
1 parent 028f4dc
commit 1cd4b9d
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 11 deletions.
diff --git a/README.md b/README.md
@@ -273,7 +273,7 @@ Here is a full list of options:
 
 ### Focus outline
 
-For accessibility reasons, `emoji-picker-element` displays a prominent focus ring. If you want to hide the focus ring for non-keyboard users (e.g. mouse and touch only), then use the [focus-visible](https://github.com/WICG/focus-visible) polyfill, e.g.:
+For accessibility reasons, `emoji-picker-element` displays a prominent focus ring for keyboard users. This uses [`:focus-visible`](https://developer.mozilla.org/en-US/docs/Web/CSS/:focus-visible) under the hood. To properly support [browsers that do not support `:focus-visible`](https://caniuse.com/css-focus-visible), you can use the [focus-visible](https://github.com/WICG/focus-visible) polyfill, e.g.:
 
 ```js
 import 'focus-visible';

diff --git a/package.json b/package.json
@@ -95,7 +95,6 @@
     "fast-glob": "^3.3.2",
     "fetch-mock": "^9.11.0",
     "flat-color-icons": "^1.1.0",
-    "focus-visible": "^5.2.0",
     "get-folder-size": "^4.0.0",
     "husky": "^9.0.11",
     "jsdom": "^24.0.0",

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/test/adhoc/index.html b/test/adhoc/index.html
@@ -47,7 +47,6 @@
 </head>
 <body>
 <button class="delete">Delete database</button>
-<script src="/node_modules/focus-visible/dist/focus-visible.js"></script>
 <script type="module">
   import { Picker, Database } from '/index.js'
   (async () => {
@@ -89,7 +88,6 @@
     }
     const picker = new Picker(opts)
     picker.addEventListener('emoji-click', e => console.log(e))
-    applyFocusVisiblePolyfill(picker.shadowRoot)
     document.body.appendChild(picker)
   })()
 </script>