fix: disable pressable content in radio and checkbox #55

Coses: #55
lad-tech · Feb 1, 2024 · c769411 · c769411
1 parent fb08cd1
commit c769411
Show file tree

Hide file tree

Showing 6 changed files with 23 additions and 19 deletions.
diff --git a/packages/core/src/controls/CheckBox/CheckBox.tsx b/packages/core/src/controls/CheckBox/CheckBox.tsx
@@ -22,7 +22,10 @@ const CheckBox: FC<IControlProps & IPressableProps> = ({children, ...rest}) => {
   const [styles] = useStyles(stylesCreate, disabled, selected, width, height);
 
   return (
-    <Pressable style={[styles.container, containerStyle]} onPress={onPress}>
+    <Pressable
+      style={[styles.container, containerStyle]}
+      onPress={onPress}
+      disabled={disabled}>
       <Control
         type={ControlType.checkBox}
         style={[styles.checkbox, checkboxStyle]}

diff --git a/packages/core/src/controls/CheckBox/__tests__/__snapshots__/CheckBox.test.tsx.snap b/packages/core/src/controls/CheckBox/__tests__/__snapshots__/CheckBox.test.tsx.snap
@@ -204,7 +204,7 @@ exports[`CheckBox should renders correctly disabled 1`] = `
     {
       "busy": undefined,
       "checked": undefined,
-      "disabled": undefined,
+      "disabled": true,
       "expanded": undefined,
       "selected": undefined,
     }

diff --git a/packages/core/src/controls/ControlsLists/__tests__/__snapshots__/ControlsList.tsx.snap b/packages/core/src/controls/ControlsLists/__tests__/__snapshots__/ControlsList.tsx.snap
@@ -16,7 +16,7 @@ exports[`ControlsList should renders correctly with checkbox type 1`] = `
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }
@@ -121,7 +121,7 @@ exports[`ControlsList should renders correctly with checkbox type 1`] = `
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }
@@ -240,7 +240,7 @@ exports[`ControlsList should renders correctly with checkbox type and horizontal
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }
@@ -345,7 +345,7 @@ exports[`ControlsList should renders correctly with checkbox type and horizontal
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }
@@ -464,7 +464,7 @@ exports[`ControlsList should renders correctly with checkbox type and single 1`]
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }
@@ -569,7 +569,7 @@ exports[`ControlsList should renders correctly with checkbox type and single 1`]
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }
@@ -688,7 +688,7 @@ exports[`ControlsList should renders correctly with radio type 1`] = `
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }
@@ -805,7 +805,7 @@ exports[`ControlsList should renders correctly with radio type 1`] = `
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }
@@ -924,7 +924,7 @@ exports[`ControlsList should renders correctly with radio type and horizontal 1`
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }
@@ -1029,7 +1029,7 @@ exports[`ControlsList should renders correctly with radio type and horizontal 1`
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }
@@ -1148,7 +1148,7 @@ exports[`ControlsList should renders correctly with radio type and single 1`] =
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }
@@ -1253,7 +1253,7 @@ exports[`ControlsList should renders correctly with radio type and single 1`] =
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }

diff --git a/packages/core/src/controls/Radio/Radio.tsx b/packages/core/src/controls/Radio/Radio.tsx
@@ -22,6 +22,7 @@ const Radio: FC<IControlProps & IRadioStyle> = ({
     <Pressable
       style={[styles.container, containerStyle]}
       needsOffscreenAlphaCompositing={true}
+      disabled={disabled}
       onPress={onPress}>
       <Control
         disabled={disabled}

diff --git a/packages/core/src/controls/Radio/__tests__/__snapshots__/Radio.test.tsx.snap b/packages/core/src/controls/Radio/__tests__/__snapshots__/Radio.test.tsx.snap
@@ -6,7 +6,7 @@ exports[`@lad-tech/mobydick-core/Radio renders correctly 1`] = `
     {
       "busy": undefined,
       "checked": undefined,
-      "disabled": undefined,
+      "disabled": false,
       "expanded": undefined,
       "selected": undefined,
     }
@@ -97,7 +97,7 @@ exports[`@lad-tech/mobydick-core/Radio renders correctly disabled 1`] = `
     {
       "busy": undefined,
       "checked": undefined,
-      "disabled": undefined,
+      "disabled": true,
       "expanded": undefined,
       "selected": undefined,
     }
@@ -188,7 +188,7 @@ exports[`@lad-tech/mobydick-core/Radio renders correctly selected 1`] = `
     {
       "busy": undefined,
       "checked": undefined,
-      "disabled": undefined,
+      "disabled": false,
       "expanded": undefined,
       "selected": undefined,
     }
@@ -291,7 +291,7 @@ exports[`@lad-tech/mobydick-core/Radio renders correctly selected and disabled 1
     {
       "busy": undefined,
       "checked": undefined,
-      "disabled": undefined,
+      "disabled": true,
       "expanded": undefined,
       "selected": undefined,
     }

diff --git a/...ges/core/src/popups/components/ActionSheetBase/__tests__/__snapshots__/Item.test.tsx.snap b/...ges/core/src/popups/components/ActionSheetBase/__tests__/__snapshots__/Item.test.tsx.snap
@@ -254,7 +254,7 @@ exports[`@lad-tech/mobydick-core/ActionSheetBase should renders correctly Item w
       {
         "busy": undefined,
         "checked": undefined,
-        "disabled": undefined,
+        "disabled": false,
         "expanded": undefined,
         "selected": undefined,
       }