canvas-kit icon indicating copy to clipboard operation
canvas-kit copied to clipboard

Modal - full width and full height support

Open NicholasBoll opened this issue 5 years ago • 2 comments

🚀 Feature Proposal

Support full-screen growth of modals. The design specification lists out how Modals should grow both in width and it height. Modals only have set sizes.

Motivation

Teams sometimes need modals that maximize content shown to minimize scrolling.

Example

Large: Screen width of 1024px or greater:

Screen Shot 2020-07-21 at 9 20 41 AM

Medium: 768px to 1024px

Screen Shot 2020-07-21 at 9 22 11 AM

Small: Less than 320px

Screen Shot 2020-07-21 at 9 22 19 AM

https://www.figma.com/file/IjDja6ooe8B4JTiMz8VJU4

NicholasBoll avatar Jul 21 '20 15:07 NicholasBoll

Related: #790, #681

NicholasBoll avatar Aug 17 '20 21:08 NicholasBoll

I ran into issues with the different margins around the different screen sizes. The modals are centered, but having a different top and bottom margin throw off the center calculations. Should Modals center or pin to the top? Why have a larger left/right margin on the medium screen size? This leaves less room for content on a smaller screen than the large screen size.

NicholasBoll avatar Jan 25 '22 23:01 NicholasBoll

Modals now grow to fit contents with a maximum size. So functionally, this is solved.

What's left for a question is margins via design.

NicholasBoll avatar Sep 20 '22 19:09 NicholasBoll

Margins via design will be handled. Usage guidance for margins will be used.

jaclynjessup avatar Oct 19 '22 19:10 jaclynjessup