Skip to content

Commit

Permalink
add note highlighting isFullScreen taking precedence
Browse files Browse the repository at this point in the history
  • Loading branch information
chad1008 committed Sep 29, 2023
1 parent d630574 commit 14669de
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion packages/components/src/modal/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -223,7 +223,7 @@ This property when set to `true` will render a full screen modal.

#### `size`: `'small' | 'medium' | 'large' | 'fill'`

If this property is added it will cause the modal to render at a preset width, or expand to fill the screen.
If this property is added it will cause the modal to render at a preset width, or expand to fill the screen. This prop will be ignored if `isFullScreen` is set to `true`.

- Required: No

Expand Down
3 changes: 2 additions & 1 deletion packages/components/src/modal/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,8 @@ export type ModalProps = {
isFullScreen?: boolean;
/**
* If this property is added it will cause the modal to render at a preset
* width, or expand to fill the screen.
* width, or expand to fill the screen. This prop will be ignored if
* `isFullScreen` is set to `true`.
*
* Note: `Modal`'s width can also be controlled by adjusting the width of the
* modal's contents, or via CSS using the `style` prop.
Expand Down

0 comments on commit 14669de

Please sign in to comment.