IconButton: make onClick required & fix modal close issue

Cambly · Mar 26, 2024 · 16dbdc1 · 16dbdc1
1 parent 9ecadcb
commit 16dbdc1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/.changeset/ninety-rice-repair.md b/.changeset/ninety-rice-repair.md
@@ -0,0 +1,5 @@
+---
+"@cambly/syntax-core": minor
+---
+
+IconButton: make onClick required & fix modal close issue
diff --git a/packages/syntax-core/src/Modal/Modal.tsx b/packages/syntax-core/src/Modal/Modal.tsx
@@ -215,13 +215,14 @@ export default function Modal({
                       accessibilityLabel={accessibilityCloseLabel}
                       color={image ? "primary" : "tertiary"}
                       on={image ? "darkBackground" : "lightBackground"}
+                      onClick={onDismiss}
                       size="sm"
                       icon={XIconCambio}
                     />
                   </Box>
                 )}
               </Box>
-              {image && <Box maxHeight={200}>{image}</Box>}
+              {image && <Box>{image}</Box>}
               <Box
                 display="flex"
                 gap={themeName === "classic" ? 3 : 4}