vevcom / projectNext

Project Next is Omegas new website coming soon
MIT License
8 stars 0 forks source link

Chore/mobile popup #314

Closed JohanHjelsethStorstad closed 2 months ago

JohanHjelsethStorstad commented 2 months ago

This PR makes the PopUp better on mobile.

theodorklauritzen commented 2 months ago

On landscape on mobile the close button is not visible. And if the pop up has a lot of content maybe a overflow-y: auto is smart, to enable scroll.

JohanHjelsethStorstad commented 2 months ago

On landscape on mobile the close button is not visible. And if the pop up has a lot of content maybe a overflow-y: auto is smart, to enable scroll.

I could not reproduce the problem with the close button. But what I can do is set max-height: 90svh on the popup content

JohanHjelsethStorstad commented 2 months ago

I guess you mean overflow: scroll

JohanHjelsethStorstad commented 2 months ago

@Paulijuz can you approve this?