feat: size support for alert-dialog #6051
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 Description
Adds
sizeprop support toAlertDialog.Containercomponent, allowing users to control the dialog width with predefined size variants:xs,sm,md,lg, andcover. This implementation follows the same pattern as theModalcomponent for consistency.⛳️ Current behavior (updates)
AlertDialog.Containeronly supportsplacementprop🚀 New behavior
AlertDialog.Containernow accepts asizeprop with values:"xs" | "sm" | "md" | "lg" | "cover""md"(matches Modal behavior)xs:max-w-xs(20rem / 320px)sm:max-w-sm(24rem / 384px)md:max-w-md(28rem / 448px) - defaultlg:max-w-lg(32rem / 512px)cover: Full height and width with marginsfullsize variant is not included as alert dialogs don't require fullscreen mode💣 Is this a breaking change (Yes/No):
No.
📝 Additional Information
Modalcomponent for consistencyalert-dialog__dialog--xs)"md"if not specified