focus-trap - npm Package Compare versions

Comparing version 0.3.0 to 0.3.1

index.js

@@ -23,11 +23,19 @@ var tabbable = require('tabbable');
   var focusNode = (function() {
-    if (!config.initialFocus) return tabbableNodes[0];
+    var node;
+    if (!config.initialFocus) {
+      node = tabbableNodes[0];
+      if (!node) {
+        throw new Error('You can\'t have a focus-trap without at least one focusable element');
+      }
+    }
     if (typeof config.initialFocus === 'string') {
-      return document.querySelector(config.initialFocus);
+      node = document.querySelector(config.initialFocus);
+    } else {
+      node = config.initialFocus;
     }
-    return config.initialFocus;
+    if (!node) {
+      throw new Error('The `initialFocus` selector you passed refers to no known node');
+    }
+    return node;
   }());
-  if (!focusNode) {
-    throw new Error('You can\'t have a focus-trap without at least one focusable element');
-  }
   focusNode.focus();
@@ -34,0 +42,0 @@

package.json

 {
   "name": "focus-trap",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Trap focus within a DOM node.",
@@ -5,0 +5,0 @@ "main": "index.js",

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

9236

increased by2.5%

Lines of code

94

increased by9.3%

No dependency changes