@oddbird/popover-polyfill - npm Package Compare versions

Comparing version 0.4.4 to 0.5.0

package.json

 {
   "name": "@oddbird/popover-polyfill",
-  "version": "0.4.4",
+  "version": "0.5.0",
   "description": "Popover Attribute Polyfill",
@@ -22,3 +22,3 @@ "license": "BSD-3-Clause",
   "type": "module",
-  "main": "./dist/popover-fn.js",
+  "main": "./dist/popover.js",
   "module": "./dist/popover.js",
@@ -37,3 +37,4 @@ "browser": "./dist/popover.iife.min.js",
       "types": "./dist/index-fn.d.ts",
-      "import": "./dist/popover-fn.js"
+      "import": "./dist/popover-fn.js",
+      "require": "./dist/popover-fn.cjs.js"
     }
@@ -65,2 +66,3 @@ },
     "build:esm-fn": "esbuild --bundle src/index-fn.ts --outfile=dist/popover-fn.js --format=esm --sourcemap",
+    "build:cjs-fn": "esbuild --bundle src/index-fn.ts --outfile=dist/popover-fn.cjs.js --format=cjs --sourcemap",
     "clean": "rm -rf dist",
@@ -90,18 +92,19 @@ "build": "run-s clean build:* types",
   "devDependencies": {
+    "@eslint/js": "^9.12.0",
     "@playwright/test": "1.33.0",
+    "@types/eslint__js": "^8.42.3",
+    "@types/eslint-config-prettier": "^6.11.3",
     "@types/node": "*",
-    "@typescript-eslint/eslint-plugin": "^7.17.0",
-    "@typescript-eslint/parser": "^7.17.0",
-    "esbuild": "^0.23.0",
-    "eslint": "^8.57.0",
+    "esbuild": "^0.24.0",
+    "eslint": "^9.12.0",
     "eslint-config-prettier": "^9.1.0",
-    "eslint-import-resolver-typescript": "^3.6.1",
-    "eslint-plugin-import": "^2.29.1",
+    "eslint-import-resolver-typescript": "^3.6.3",
+    "eslint-plugin-import": "^2.31.0",
     "eslint-plugin-simple-import-sort": "^12.1.1",
-    "husky": "^9.1.1",
-    "lint-staged": "^15.2.7",
+    "husky": "^9.1.6",
+    "lint-staged": "^15.2.10",
     "npm-run-all": "^4.1.5",
     "prettier": "3.3.3",
-    "ts-node": "^10.9.2",
-    "typescript": "^5.5.3"
+    "typescript": "^5.6.2",
+    "typescript-eslint": "^8.8.1"
   },
@@ -108,0 +111,0 @@ "lint-staged": {

README.md

@@ -64,5 +64,12 @@ # Popover Attribute Polyfill
 
-A special `isPolyfilled` function is also available, to detect if the Popover methods have been polyfilled:
+Or in CommonJS environments:
 
 ```js
+const { apply, isSupported } = require('@oddbird/popover-polyfill/fn');
+```
+
+An `isPolyfilled` function is also available, to detect if the Popover methods
+have been polyfilled:
+
+```js
 import { isPolyfilled } from '@oddbird/popover-polyfill/fn';
@@ -69,0 +76,0 @@ ```

src/index-fn.ts

@@ -28,6 +28,6 @@ interface PopoverToggleTargetElementInvoker {
 
-  /* eslint-disable @typescript-eslint/no-empty-interface */
+  /* eslint-disable @typescript-eslint/no-empty-object-type */
   interface HTMLButtonElement extends PopoverToggleTargetElementInvoker {}
   interface HTMLInputElement extends PopoverToggleTargetElementInvoker {}
-  /* eslint-enable @typescript-eslint/no-empty-interface */
+  /* eslint-enable @typescript-eslint/no-empty-object-type */
 
@@ -34,0 +34,0 @@ interface Window {

src/index.ts

@@ -30,6 +30,6 @@ import { apply, isSupported } from './popover.js';
 
-  /* eslint-disable @typescript-eslint/no-empty-interface */
+  /* eslint-disable @typescript-eslint/no-empty-object-type */
   interface HTMLButtonElement extends PopoverToggleTargetElementInvoker {}
   interface HTMLInputElement extends PopoverToggleTargetElementInvoker {}
-  /* eslint-enable @typescript-eslint/no-empty-interface */
+  /* eslint-enable @typescript-eslint/no-empty-object-type */
 
@@ -36,0 +36,0 @@ interface Window {

src/popover-helpers.ts

@@ -87,3 +87,2 @@ import { queuePopoverToggleEventTask, ToggleEvent } from './events.js';
     if (!popover.isConnected) {
-      // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
       documentPopovers!.delete(popover);
@@ -289,3 +288,2 @@ } else {
   }
-  // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
   topLayerElements.get(document)!.add(element);
@@ -297,3 +295,2 @@ popoverFocusingSteps(element);
     }
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     autoPopoverList.get(document)!.add(element);
@@ -300,0 +297,0 @@ setInvokerAriaExpanded(popoverInvoker.get(element), true);

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

330833

increased by23.09%

Number of package files

25

increased by8.7%

Lines of code

3112

increased by27.86%

Number of lines in readme file

194

increased by3.74%

Worsened metrics

Dev dependency count

17

increased by6.25%

No dependency changes