@jsenv/navi 0.30.3 → 0.30.4

package/dist/dev/jsenv_navi.js

@@ -11751,7 +11751,7 @@ const getMessageString = (message) => {
   if (typeof message === "string") {
     return message;
   }
-  if (Error.isError(message)) {
+  if (message instanceof Error) {
     return message.message;
   }
   return undefined;
@@ -20281,7 +20281,7 @@ const openCallout = (
         element: calloutElement,
       });
     } else {
-      if (Error.isError(newMessage)) {
+      if (newMessage instanceof Error) {
         const error = newMessage;
         newMessage = error.message;
         if (showErrorStack && error.stack) {
@@ -23931,7 +23931,7 @@ const sayRefusal = (element, error) => {
   let anchorElement = element;
   if (
     typeof error === "object" &&
-    !Error.isError(error) &&
+    !(error instanceof Error) &&
     !isValidElement(error)
   ) {
     message = error.message;
@@ -26739,7 +26739,7 @@ const useExecuteAction = (
       //   and it is then shown exactly as if there were no mapping at all.
       if (typeof errorMappingResult === "string") {
         message = errorMappingResult;
-      } else if (Error.isError(errorMappingResult)) {
+      } else if (errorMappingResult instanceof Error) {
         message = errorMappingResult;
       } else if (isValidElement(errorMappingResult)) {
         message = errorMappingResult;
@@ -84981,7 +84981,7 @@ const css$w = /* css */`@property --list-group-label-height {
   overflow: clip;
 }
 
-.navi_list_container[data-horizontal]:not([data-scroller], [data-overflow-visible]) > .navi_list_scroll_container > .navi_list:not([navi-box-flow="grid"]) {
+.navi_list_container[data-horizontal]:not([data-scroller], [data-overflow-visible], [data-flex-wrap]) > .navi_list_scroll_container > .navi_list:not([navi-box-flow="grid"]) {
   width: min-content;
   min-width: 100%;
 }
@@ -85542,7 +85542,7 @@ const ListUI = props => {
   // rule (min-content, never narrower than its box — see the css), which
   // already gives it what expandX asks. An inline width from expandX would beat
   // that rule and lay the end room among the items again.
-  const trackSizedByItsRule = horizontal && !scrollerAttribute && !overflowVisibleAttribute && !(columns || itemColumns);
+  const trackSizedByItsRule = horizontal && !scrollerAttribute && !overflowVisibleAttribute && !rest.flexWrap && !(columns || itemColumns);
   return jsx(Box, {
     ...rest,
     ref: ref,
@@ -85553,6 +85553,7 @@ const ListUI = props => {
     "data-borderless": isBorderless(rest) ? "" : undefined,
     "data-scroller": scrollerAttribute,
     "data-overflow-visible": overflowVisibleAttribute,
+    "data-flex-wrap": rest.flexWrap ? "" : undefined,
     "navi-hover-while-scrolling": hoverWhileScrolling ? "" : undefined,
     "data-expand-x": expandX || expand ? "" : undefined,
     "data-expand-y": expandY || expand ? "" : undefined,
@@ -86425,6 +86426,12 @@ const useListScrollSync = ({
   // (pour l'instant pas grave car on travaille pour le mode select qui fermera le dialog au select)
   const topMatchScoresKeyRef = useRef("");
   const restoreScrollRafRef = useRef(null);
+  // The frame reads the list's ref, which an unmount empties.
+  useLayoutEffect(() => {
+    return () => {
+      cancelAnimationFrame(restoreScrollRafRef.current);
+    };
+  }, []);
   useLayoutEffect(() => {
     const listScrollContainerEl = ref.current ? getScroller() : null;
     if (!listScrollContainerEl) {
@@ -86463,10 +86470,9 @@ const useListScrollSync = ({
         // based on virtual item height which can wrongly restore the scroll.
         // However we have a contract with outside to inside which item is scrolled
         // (used by keyboard nav to enable anchoring the item for list item nav with arrow keys)
-        // so we do our best to give that item back
-        const {
-          item
-        } = getScrollInfo({
+        // so we do our best to give that item back: there is none to give when
+        // the list has no visible part, and the restore happens all the same.
+        const scrollInfo = getScrollInfo({
           scrollValues: savedScroll,
           scrollerEl: listScrollContainerEl,
           listEl: getListEl(),
@@ -86482,7 +86488,7 @@ const useListScrollSync = ({
         });
         const listEl = getListEl();
         dispatchPublicCustomEvent(listEl, "navi_scroll", {
-          item,
+          item: scrollInfo?.item,
           event: new CustomEvent("navi_scroll_restore")
         });
       });