Skip to content

Commit 417ca66

Browse files
docs(dialog): remove custom modal style
- removes the customStyles object that was setting --mod-modal- background-color in the dialog template - removes the custom wrapper styles in the dialog tests that were intending to set the dialog apart from the modal - remove a duplicate isOpen control in dialog.stories.js
1 parent bf4431b commit 417ca66

File tree

3 files changed

+0
-8
lines changed

3 files changed

+0
-8
lines changed

components/dialog/stories/dialog.stories.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,6 @@ export default {
9090
options: ["default", "fullscreen", "fullscreenTakeover"],
9191
control: "select",
9292
},
93-
isOpen,
9493
isDismissible: {
9594
name: "Dismissible",
9695
type: { name: "boolean" },

components/dialog/stories/dialog.test.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,6 @@ export const DialogGroup = Variants({
1616
// TODO: The dialog's heading arg is getting passed as the "Sizing" heading arg (instead of the
1717
// TODO: word "Sizing"). We should be able to remove this arg once that no longers happens.
1818
heading: showTestingGrid ? "Lorem ipsum dolor sit amet, consectetur adipiscing elit" : args.heading,
19-
customStyles: {
20-
...(args.customStyles ?? {}),
21-
"background-color": showTestingGrid ? "var(--spectrum-gray-100)" : undefined,
22-
},
2319
}, context);
2420
},
2521
sizeDirection: "column",

components/dialog/stories/template.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -171,9 +171,6 @@ export const Template = ({
171171
isOpen,
172172
content: Dialog,
173173
variant: layout,
174-
customStyles: {
175-
"--mod-modal-background-color": "transparent"
176-
},
177174
}, context);
178175
}
179176
else {

0 commit comments

Comments
 (0)