fix(baseModal): add "smaller-h-full" size option to improve modal size flexibility
fix(exportModal): update size prop of BaseModal to "smaller-h-full" to match the new size option
This commit is contained in:
parent
dd63dadd51
commit
4b8349d131
2 changed files with 8 additions and 3 deletions
|
|
@ -67,7 +67,8 @@ interface BaseModalProps {
|
||||||
| "large"
|
| "large"
|
||||||
| "large-h-full"
|
| "large-h-full"
|
||||||
| "small-h-full"
|
| "small-h-full"
|
||||||
| "medium-h-full";
|
| "medium-h-full"
|
||||||
|
| "smaller-h-full";
|
||||||
|
|
||||||
disable?: boolean;
|
disable?: boolean;
|
||||||
onChangeOpenModal?: (open?: boolean) => void;
|
onChangeOpenModal?: (open?: boolean) => void;
|
||||||
|
|
@ -104,6 +105,10 @@ function BaseModal({
|
||||||
minWidth = "min-w-[40vw]";
|
minWidth = "min-w-[40vw]";
|
||||||
height = "h-[27vh]";
|
height = "h-[27vh]";
|
||||||
break;
|
break;
|
||||||
|
case "smaller-h-full":
|
||||||
|
minWidth = "min-w-[40vw]";
|
||||||
|
height = "h-full";
|
||||||
|
break;
|
||||||
case "small":
|
case "small":
|
||||||
minWidth = "min-w-[40vw]";
|
minWidth = "min-w-[40vw]";
|
||||||
height = "h-[40vh]";
|
height = "h-[40vh]";
|
||||||
|
|
|
||||||
|
|
@ -24,7 +24,7 @@ const ExportModal = forwardRef(
|
||||||
const [open, setOpen] = useState(false);
|
const [open, setOpen] = useState(false);
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<BaseModal size="smaller" open={open} setOpen={setOpen}>
|
<BaseModal size="smaller-h-full" open={open} setOpen={setOpen}>
|
||||||
<BaseModal.Trigger>{props.children}</BaseModal.Trigger>
|
<BaseModal.Trigger>{props.children}</BaseModal.Trigger>
|
||||||
<BaseModal.Header description={EXPORT_DIALOG_SUBTITLE}>
|
<BaseModal.Header description={EXPORT_DIALOG_SUBTITLE}>
|
||||||
<span className="pr-2">Export</span>
|
<span className="pr-2">Export</span>
|
||||||
|
|
@ -55,7 +55,7 @@ const ExportModal = forwardRef(
|
||||||
Save with my API keys
|
Save with my API keys
|
||||||
</label>
|
</label>
|
||||||
</div>
|
</div>
|
||||||
<span className="text-xs text-destructive">
|
<span className=" text-xs text-destructive ">
|
||||||
Caution: Uncheck this box only removes API keys from fields
|
Caution: Uncheck this box only removes API keys from fields
|
||||||
specifically designated for API keys.
|
specifically designated for API keys.
|
||||||
</span>
|
</span>
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue