@react-aria/utils - npm Package Compare versions

Comparing version 3.0.0-nightly-641446f65-240905 to 3.0.0-nightly-69109f003-241018

dist/openLink.main.js

@@ -123,4 +123,6 @@ var $1117b6c0d4c4c164$exports = require("./focusWithoutScrolling.main.js");
     let router = $4068a0fae83b6d84$export$9a302a45f65d0572();
+    var _props_href;
+    const href = router.useHref((_props_href = props.href) !== null && _props_href !== void 0 ? _props_href : '');
     return {
-        'data-href': props.href ? router.useHref(props.href) : undefined,
+        'data-href': props.href ? href : undefined,
         'data-target': props.target,
@@ -145,4 +147,6 @@ 'data-rel': props.rel,
     let router = $4068a0fae83b6d84$export$9a302a45f65d0572();
+    var _props_href;
+    const href = router.useHref((_props_href = props === null || props === void 0 ? void 0 : props.href) !== null && _props_href !== void 0 ? _props_href : '');
     return {
-        href: (props === null || props === void 0 ? void 0 : props.href) ? router.useHref(props === null || props === void 0 ? void 0 : props.href) : undefined,
+        href: (props === null || props === void 0 ? void 0 : props.href) ? href : undefined,
         target: props === null || props === void 0 ? void 0 : props.target,
@@ -149,0 +153,0 @@ rel: props === null || props === void 0 ? void 0 : props.rel,

dist/openLink.module.js

@@ -107,4 +107,6 @@ import {focusWithoutScrolling as $7215afc6de606d6b$export$de79e2c695e052f3} from "./focusWithoutScrolling.module.js";
     let router = $ea8dcbcb9ea1b556$export$9a302a45f65d0572();
+    var _props_href;
+    const href = router.useHref((_props_href = props.href) !== null && _props_href !== void 0 ? _props_href : '');
     return {
-        'data-href': props.href ? router.useHref(props.href) : undefined,
+        'data-href': props.href ? href : undefined,
         'data-target': props.target,
@@ -129,4 +131,6 @@ 'data-rel': props.rel,
     let router = $ea8dcbcb9ea1b556$export$9a302a45f65d0572();
+    var _props_href;
+    const href = router.useHref((_props_href = props === null || props === void 0 ? void 0 : props.href) !== null && _props_href !== void 0 ? _props_href : '');
     return {
-        href: (props === null || props === void 0 ? void 0 : props.href) ? router.useHref(props === null || props === void 0 ? void 0 : props.href) : undefined,
+        href: (props === null || props === void 0 ? void 0 : props.href) ? href : undefined,
         target: props === null || props === void 0 ? void 0 : props.target,
@@ -133,0 +137,0 @@ rel: props === null || props === void 0 ? void 0 : props.rel,

dist/types.d.ts

@@ -224,3 +224,3 @@ import React, { ForwardedRef, MutableRefObject, ReactNode, HTMLAttributes, EffectCallback, Dispatch, RefObject as _RefObject1 } from "react";
     /** The data currently loaded. */
-    items?: any[];
+    items?: any;
 }
@@ -227,0 +227,0 @@ export function useLoadMore(props: LoadMoreProps, ref: _RefObject1<HTMLElement | null>): void;

dist/useId.main.js

@@ -38,3 +38,12 @@ var $78605a5d7424e31b$exports = require("./useLayoutEffect.main.js");
     }, []);
-    if ($8c61827343eed941$var$canUseDOM) $8c61827343eed941$var$idsUpdaterMap.set(res, updateValue);
+    if ($8c61827343eed941$var$canUseDOM) {
+        // TS not smart enough to know that `has` means the value exists
+        if ($8c61827343eed941$var$idsUpdaterMap.has(res) && !$8c61827343eed941$var$idsUpdaterMap.get(res).includes(updateValue)) $8c61827343eed941$var$idsUpdaterMap.set(res, [
+            ...$8c61827343eed941$var$idsUpdaterMap.get(res),
+            updateValue
+        ]);
+        else $8c61827343eed941$var$idsUpdaterMap.set(res, [
+            updateValue
+        ]);
+    }
     (0, $78605a5d7424e31b$exports.useLayoutEffect)(()=>{
@@ -61,10 +70,10 @@ let r = res;
     if (idA === idB) return idA;
-    let setIdA = $8c61827343eed941$var$idsUpdaterMap.get(idA);
-    if (setIdA) {
-        setIdA(idB);
+    let setIdsA = $8c61827343eed941$var$idsUpdaterMap.get(idA);
+    if (setIdsA) {
+        setIdsA.forEach((fn)=>fn(idB));
         return idB;
     }
-    let setIdB = $8c61827343eed941$var$idsUpdaterMap.get(idB);
-    if (setIdB) {
-        setIdB(idA);
+    let setIdsB = $8c61827343eed941$var$idsUpdaterMap.get(idB);
+    if (setIdsB) {
+        setIdsB.forEach((fn)=>fn(idA));
         return idA;
@@ -71,0 +80,0 @@ }

dist/useId.module.js

@@ -30,3 +30,12 @@ import {useLayoutEffect as $f0a04ccd8dbdd83b$export$e5c5a5f917a5871c} from "./useLayoutEffect.module.js";
     }, []);
-    if ($bdb11010cef70236$var$canUseDOM) $bdb11010cef70236$var$idsUpdaterMap.set(res, updateValue);
+    if ($bdb11010cef70236$var$canUseDOM) {
+        // TS not smart enough to know that `has` means the value exists
+        if ($bdb11010cef70236$var$idsUpdaterMap.has(res) && !$bdb11010cef70236$var$idsUpdaterMap.get(res).includes(updateValue)) $bdb11010cef70236$var$idsUpdaterMap.set(res, [
+            ...$bdb11010cef70236$var$idsUpdaterMap.get(res),
+            updateValue
+        ]);
+        else $bdb11010cef70236$var$idsUpdaterMap.set(res, [
+            updateValue
+        ]);
+    }
     (0, $f0a04ccd8dbdd83b$export$e5c5a5f917a5871c)(()=>{
@@ -53,10 +62,10 @@ let r = res;
     if (idA === idB) return idA;
-    let setIdA = $bdb11010cef70236$var$idsUpdaterMap.get(idA);
-    if (setIdA) {
-        setIdA(idB);
+    let setIdsA = $bdb11010cef70236$var$idsUpdaterMap.get(idA);
+    if (setIdsA) {
+        setIdsA.forEach((fn)=>fn(idB));
         return idB;
     }
-    let setIdB = $bdb11010cef70236$var$idsUpdaterMap.get(idB);
-    if (setIdB) {
-        setIdB(idA);
+    let setIdsB = $bdb11010cef70236$var$idsUpdaterMap.get(idB);
+    if (setIdsB) {
+        setIdsB.forEach((fn)=>fn(idA));
         return idA;
@@ -63,0 +72,0 @@ }

dist/useLoadMore.main.js

@@ -63,3 +63,4 @@ var $2a8c0bb1629926c8$exports = require("./useEvent.main.js");
         props,
-        ref
+        ref,
+        items
     ]);
@@ -66,0 +67,0 @@ // TODO: maybe this should still just return scroll props?

dist/useLoadMore.module.js

@@ -57,3 +57,4 @@ import {useEvent as $e9faafb641e167db$export$90fc3a17d93f704c} from "./useEvent.module.js";
         props,
-        ref
+        ref,
+        items
     ]);
@@ -60,0 +61,0 @@ // TODO: maybe this should still just return scroll props?

package.json

 {
   "name": "@react-aria/utils",
-  "version": "3.0.0-nightly-641446f65-240905",
+  "version": "3.0.0-nightly-69109f003-241018",
   "description": "Spectrum UI components in React",
@@ -25,5 +25,5 @@ "license": "Apache-2.0",
   "dependencies": {
-    "@react-aria/ssr": "^3.0.0-nightly-641446f65-240905",
-    "@react-stately/utils": "^3.0.0-nightly-641446f65-240905",
-    "@react-types/shared": "^3.0.0-nightly-641446f65-240905",
+    "@react-aria/ssr": "^3.0.0-nightly-69109f003-241018",
+    "@react-stately/utils": "^3.0.0-nightly-69109f003-241018",
+    "@react-types/shared": "^3.0.0-nightly-69109f003-241018",
     "@swc/helpers": "^0.5.0",
@@ -38,3 +38,3 @@ "clsx": "^2.0.0"
   },
-  "stableVersion": "3.25.2"
+  "stableVersion": "3.25.3"
 }

src/useId.ts

@@ -25,3 +25,3 @@ /*
 
-let idsUpdaterMap: Map<string, (v: string) => void> = new Map();
+let idsUpdaterMap: Map<string, Array<(v: string) => void>> = new Map();
 
@@ -43,3 +43,8 @@ /**
   if (canUseDOM) {
-    idsUpdaterMap.set(res, updateValue);
+    // TS not smart enough to know that `has` means the value exists
+    if (idsUpdaterMap.has(res) && !idsUpdaterMap.get(res)!.includes(updateValue)) {
+      idsUpdaterMap.set(res, [...idsUpdaterMap.get(res)!, updateValue]);
+    } else {
+      idsUpdaterMap.set(res, [updateValue]);
+    }
   }
@@ -76,11 +81,11 @@
 
-  let setIdA = idsUpdaterMap.get(idA);
-  if (setIdA) {
-    setIdA(idB);
+  let setIdsA = idsUpdaterMap.get(idA);
+  if (setIdsA) {
+    setIdsA.forEach(fn => fn(idB));
     return idB;
   }
 
-  let setIdB = idsUpdaterMap.get(idB);
-  if (setIdB) {
-    setIdB(idA);
+  let setIdsB = idsUpdaterMap.get(idB);
+  if (setIdsB) {
+    setIdsB.forEach(fn => fn(idA));
     return idA;
@@ -87,0 +92,0 @@ }

src/useLoadMore.ts

@@ -32,3 +32,3 @@ /*
   /** The data currently loaded. */
-  items?: any[]
+  items?: any
 }
@@ -77,3 +77,3 @@
     lastItems.current = items;
-  }, [isLoading, onLoadMore, props, ref]);
+  }, [isLoading, onLoadMore, props, ref, items]);
 
@@ -80,0 +80,0 @@ // TODO: maybe this should still just return scroll props?

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

559496

increased by0.67%

Lines of code

6597

increased by0.75%

Dependency changes

• Updated@react-aria/ssr@^3.0.0-nightly-69109f003-241018

• Updated@react-stately/utils@^3.0.0-nightly-69109f003-241018

• Updated@react-types/shared@^3.0.0-nightly-69109f003-241018