coles-solid-library 0.3.8 → 0.3.10

package/dist/index.esm.js

@@ -599,8 +599,15 @@ const Carousel = ({
   const [internalIndex, setInternalIndex] = createSignal(startingIndex);
   const [propIndex, other] = splitProps(props, ["currentIndex"]);
   const [, setCarouselBodyRef] = createSignal(null);
+  // Unified read/write for controlled vs uncontrolled usage
   const getPropIndex = () => propIndex.currentIndex ? propIndex.currentIndex[0]() : internalIndex();
-  const setPropIndex = index => propIndex.currentIndex ? propIndex.currentIndex[1](index) : setInternalIndex(index);
+  const setPropIndex = index => {
+    if (propIndex.currentIndex) {
+      propIndex.currentIndex[1](index);
+    } else {
+      setInternalIndex(index);
+    }
+  };
   const [currSlideName] = useContext(carouselTitleContext);
   // ----- Memoized Values -----
   const getElementsLength = createMemo(() => {
@@ -625,12 +632,8 @@ const Carousel = ({
       return [theProps.children];
     }
   });
-  const currentIndex = createMemo(() => {
-    if (getElementsLength() === 2) {
-      return getPropIndex();
-    }
-    return internalIndex();
-  });
+  // The active index should always come from external control if provided
+  const currentIndex = createMemo(() => getPropIndex());
   const slideName = createMemo(() => {
     if (Object.keys(props).includes('elements') && getElementsLength() > 0) {
       return getElementsArray()[currentIndex()]?.name;
@@ -659,23 +662,20 @@ const Carousel = ({
   });
   // ----- Functions -----
   const nextSlide = () => {
-    if (currentIndex() !== getElementsLength() - 1) {
-      setInternalIndex(old => old + 1);
-    } else {
-      setInternalIndex(0);
-    }
+    const next = currentIndex() !== getElementsLength() - 1 ? currentIndex() + 1 : 0;
+    setPropIndex(next);
   };
   const prevSlide = () => {
-    if (currentIndex() !== 0) {
-      setInternalIndex(old => old - 1);
-    } else {
-      setInternalIndex(getElementsLength() - 1);
-    }
+    const prev = currentIndex() !== 0 ? currentIndex() - 1 : getElementsLength() - 1;
+    setPropIndex(prev);
   };
   // ----- Effects -----
+  // Keep internal signal in sync when switching from uncontrolled to controlled or external changes
   createRenderEffect(() => {
-    if (props.currentIndex) {
-      setPropIndex(currentIndex());
+    if (!propIndex.currentIndex) return; // controlled only
+    // Mirror external index into internal for consistency (e.g., if component logic elsewhere reads internalIndex)
+    if (internalIndex() !== propIndex.currentIndex[0]()) {
+      setInternalIndex(propIndex.currentIndex[0]());
     }
   });
   // ----- TSX -----
@@ -1637,31 +1637,24 @@ function Checkbox(props) {
     const target = e.currentTarget;
     const newChecked = target.checked;
     if (props.checked !== undefined) {
-      // controlled: revert DOM change and just emit
-      e.preventDefault();
-      queueMicrotask(() => {
-        target.checked = !!props.checked;
-      });
+      // controlled: emit change; parent should update props.checked.
+      // After microtask (allow parent reactive update) sync DOM to current checkedState
       props.onChange?.(newChecked);
-    } else {
-      commitValue(newChecked);
-    }
-  };
-  const handleClick = e => {
-    if (props.checked !== undefined) {
-      e.preventDefault();
-      const target = e.currentTarget;
       queueMicrotask(() => {
-        target.checked = !!props.checked;
+        // checkedState() reflects latest props / form context
+        const desired = checkedState();
+        if (target.checked !== desired) target.checked = desired;
       });
+      return;
     }
+    // uncontrolled / form-managed
+    commitValue(newChecked);
   };
   return (() => {
     var _el$ = _tmpl$$h(),
       _el$2 = _el$.firstChild,
       _el$3 = _el$2.nextSibling;
     _el$2.addEventListener("change", handleChange);
-    _el$2.$$click = handleClick;
     insert(_el$, (() => {
       var _c$ = createMemo(() => !!props.label);
       return () => _c$() && (() => {
@@ -1700,7 +1693,6 @@ function Checkbox(props) {
     return _el$;
   })();
 }
-delegateEvents(["click"]);
 
 var css_248z$c = ".selectStyles-module_solid_select__placeholder__VO5-- {\n  opacity: var(--text-emphasis-medium, 87%);\n  transition: all 0.7s ease;\n}\n\n.selectStyles-module_solid_select__transparent__nOpvm {\n  background-color: transparent;\n}\n\n.selectStyles-module_solid_select__3plh1 {\n  display: inline-block;\n  position: relative;\n  background-color: var(--container-color, #ffffff);\n  color: var(--on-container-color, #000);\n  text-align: left;\n  border-radius: 8px;\n  width: 100%;\n  margin: 0px 6px;\n}\n\n.selectStyles-module_solid_select__control__Wmmpg {\n  display: flex;\n  align-items: center;\n  width: 100%;\n  max-height: 48px;\n  border-radius: 8px;\n  background-color: var(--container-color, #ffffff);\n  color: var(--on-container-color, #000);\n  cursor: pointer;\n  transition: border 0.7s ease, padding 0.7s ease;\n  overflow: hidden;\n  text-overflow: ellipsis;\n}\n.selectStyles-module_solid_select__control__Wmmpg:first-child {\n  flex-grow: 1;\n}\n.selectStyles-module_solid_select__control__Wmmpg:last-child {\n  padding: 0px 6px;\n  margin: 0px;\n}\n\n.selectStyles-module_solid_select__control_no_form__Cq9Sc {\n  display: flex;\n  align-items: center;\n  width: 100%;\n  height: 48px;\n  border-radius: 8px;\n  width: 200px;\n  margin-top: 2px;\n  cursor: pointer;\n}\n.selectStyles-module_solid_select__control_no_form__Cq9Sc:first-child {\n  flex-grow: 1;\n}\n.selectStyles-module_solid_select__control_no_form__Cq9Sc:last-child {\n  padding: 0px 6px;\n  margin: 0px;\n}\n\n.selectStyles-module_solid_select__arrow__OPCZo {\n  font-size: 0.7em;\n}\n\n.selectStyles-module_solid_select__value__rNtqF {\n  flex-grow: 1;\n  transition: all 0.3s ease;\n  white-space: nowrap;\n  overflow: hidden;\n  text-overflow: ellipsis;\n  display: block;\n}\n\n.selectStyles-module_multiselect-value__UUhhP {\n  white-space: nowrap;\n  overflow: hidden;\n  text-overflow: ellipsis;\n  max-width: 100%;\n}\n\n.selectStyles-module_solid_select__dropdown__UCt-N {\n  position: absolute;\n  background-color: var(--surface-color, #EEEEEE);\n  top: 100%;\n  left: 0;\n  z-index: 999999999999999;\n  max-height: 200px;\n  border-radius: 8px;\n  overflow-x: hidden;\n  overflow-y: auto;\n  box-shadow: 0 2px 5px rgba(0, 0, 0, 0.2);\n  transform-origin: top;\n  transform: scaleY(0);\n  transition: transform 0.3s ease-out, opacity 0.3s ease-out;\n  opacity: 0;\n}\n\n.selectStyles-module_dropTop__IVz9p {\n  transform-origin: bottom;\n}\n\n.selectStyles-module_dropBottom__cYJF0 {\n  transform-origin: top;\n}\n\n.selectStyles-module_open__f8zLA {\n  transform: scaleY(1);\n  opacity: 1;\n}\n\n.selectStyles-module_solid_select__option__47WMY {\n  padding: 8px;\n  cursor: pointer;\n  display: flex;\n  align-items: center;\n  min-height: 32px;\n  background-color: var(--container-color, #ffffff);\n  color: var(--on-container-color, #000);\n  border-radius: 8px;\n}\n.selectStyles-module_solid_select__option__47WMY:hover {\n  border: 2px inset var(--primary-color-varient, #43A047);\n}\n.selectStyles-module_solid_select__option__47WMY:has(.selectStyles-module_selected__1-CKm) {\n  font-weight: bold;\n}\n\n.selectStyles-module_checkmark__UWcbd {\n  display: inline-block;\n  width: 1.2em;\n  margin-right: 4px;\n  text-align: center;\n}\n\n.selectStyles-module_option-label__bBFSW {\n  width: -moz-max-content;\n  width: max-content;\n  height: -moz-max-content;\n  height: max-content;\n}";
 var styles$7 = {"solid_select__placeholder":"selectStyles-module_solid_select__placeholder__VO5--","solid_select__transparent":"selectStyles-module_solid_select__transparent__nOpvm","solid_select":"selectStyles-module_solid_select__3plh1","solid_select__control":"selectStyles-module_solid_select__control__Wmmpg","solid_select__control_no_form":"selectStyles-module_solid_select__control_no_form__Cq9Sc","solid_select__arrow":"selectStyles-module_solid_select__arrow__OPCZo","solid_select__value":"selectStyles-module_solid_select__value__rNtqF","multiselect-value":"selectStyles-module_multiselect-value__UUhhP","solid_select__dropdown":"selectStyles-module_solid_select__dropdown__UCt-N","dropTop":"selectStyles-module_dropTop__IVz9p","dropBottom":"selectStyles-module_dropBottom__cYJF0","open":"selectStyles-module_open__f8zLA","solid_select__option":"selectStyles-module_solid_select__option__47WMY","selected":"selectStyles-module_selected__1-CKm","checkmark":"selectStyles-module_checkmark__UWcbd","option-label":"selectStyles-module_option-label__bBFSW"};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "coles-solid-library",
-  "version": "0.3.8",
+  "version": "0.3.10",
   "description": "A SolidJS mostly UI library",
   "module": "dist/index.esm.js",
   "types": "dist/index.d.ts",