Skip to content

Commit

Permalink
fix: duplicate modal props
Browse files Browse the repository at this point in the history
  • Loading branch information
muhamien committed Feb 26, 2024
1 parent a8e75e3 commit ce1d6b1
Show file tree
Hide file tree
Showing 12 changed files with 446 additions and 62 deletions.
2 changes: 1 addition & 1 deletion packages/components/accordion/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@
"react": ">=18",
"react-dom": ">=18",
"framer-motion": ">=4.0.0",
"@jala-banyu/theme": ">=1.0.0",
"@jala-banyu/theme": "1.3.0",
"@jala-banyu/system": ">=1.0.0"
},
"dependencies": {
Expand Down
2 changes: 1 addition & 1 deletion packages/components/checkbox/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
"peerDependencies": {
"react": ">=18",
"react-dom": ">=18",
"@jala-banyu/theme": ">=1.0.0",
"@jala-banyu/theme": "1.3.0",
"@jala-banyu/system": ">=1.0.0"
},
"dependencies": {
Expand Down
2 changes: 1 addition & 1 deletion packages/components/chip/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
"peerDependencies": {
"react": ">=18",
"react-dom": ">=18",
"@jala-banyu/theme": ">=1.0.0",
"@jala-banyu/theme": "1.3.0",
"@jala-banyu/system": ">=1.0.0"
},
"dependencies": {
Expand Down
2 changes: 1 addition & 1 deletion packages/components/listbox/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
"peerDependencies": {
"react": ">=18",
"react-dom": ">=18",
"@jala-banyu/theme": ">=1.0.0",
"@jala-banyu/theme": "1.3.0",
"@jala-banyu/system": ">=1.0.0"
},
"dependencies": {
Expand Down
2 changes: 1 addition & 1 deletion packages/components/modal/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@jala-banyu/modal",
"version": "0.0.0",
"version": "0.0.1",
"description": "Displays a dialog with a custom content that requires attention or provides additional information.",
"keywords": [
"modal"
Expand Down
2 changes: 1 addition & 1 deletion packages/components/scroll-shadow/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
"peerDependencies": {
"react": ">=18",
"react-dom": ">=18",
"@jala-banyu/theme": ">=1.0.0",
"@jala-banyu/theme": "1.3.0",
"@jala-banyu/system": ">=1.0.0"
},
"dependencies": {
Expand Down
4 changes: 2 additions & 2 deletions packages/components/select/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@jala-banyu/select",
"version": "1.1.0",
"version": "0.0.0",
"description": "A select displays a collapsible list of options and allows a user to select one of them.",
"keywords": [
"select"
Expand Down Expand Up @@ -37,7 +37,7 @@
"react": ">=18",
"react-dom": ">=18",
"framer-motion": ">=4.0.0",
"@jala-banyu/theme": ">=1.0.0",
"@jala-banyu/theme": "1.3.0",
"@jala-banyu/system": ">=1.0.0"
},
"dependencies": {
Expand Down
2 changes: 1 addition & 1 deletion packages/components/spacer/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
"peerDependencies": {
"react": ">=18",
"react-dom": ">=18",
"@jala-banyu/theme": ">=1.0.0",
"@jala-banyu/theme": "1.3.0",
"@jala-banyu/system": ">=1.0.0"
},
"dependencies": {
Expand Down
13 changes: 7 additions & 6 deletions packages/core/theme/src/components/modal.ts
Original file line number Diff line number Diff line change
Expand Up @@ -157,12 +157,6 @@ const modal = tv({
base: "my-16",
},
},
isDismissable: {
true: "",
false: {
closeButton: "hidden",
},
},
},
defaultVariants: {
size: "sm",
Expand All @@ -171,6 +165,7 @@ const modal = tv({
placement: "auto",
backdrop: "opaque",
scrollBehavior: "normal",
isDismissable: true,
},
compoundVariants: [
// backdrop (opaque/blur)
Expand All @@ -180,6 +175,12 @@ const modal = tv({
backdrop: "w-screen h-screen fixed inset-0",
},
},
{
isDismissable: false,
class: {
closeButton: "hidden",
},
},
],
});

Expand Down
5 changes: 2 additions & 3 deletions packages/storybook/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,12 @@
"@jala-banyu/react": "workspace: *",
"@jala-banyu/system": "workspace: *",
"@jala-banyu/theme": "workspace: *",
"@muhamien/jala-icons": "^2.0.2",
"@storybook/addon-storysource": "^7.6.10",
"react": "^18.2.0",
"react-dom": "^18.2.0"
},
"devDependencies": {
"@muhamien/jala-icons": "^2.0.2",
"@jala-banyu/icons": "^1.0.7",
"@storybook/addon-a11y": "^7.4.6",
"@storybook/addon-actions": "^7.4.6",
"@storybook/addon-designs": "^7.0.9",
Expand All @@ -42,4 +41,4 @@
"typescript": "^5.2.2",
"vite": "^5.0.8"
}
}
}
4 changes: 2 additions & 2 deletions plop/component/package.json.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,8 @@
"peerDependencies": {
"react": ">=18",
"react-dom": ">=18"
"@jala-banyu/theme": ">=2.0.0",
"@jala-banyu/system": ">=2.0.0"
"@jala-banyu/theme": ">=1.3.0",
"@jala-banyu/system": ">=1.0.0"
},
"dependencies": {
"@jala-banyu/shared-utils": "workspace: *",
Expand Down
Loading

0 comments on commit ce1d6b1

Please sign in to comment.