Skip to content
This repository has been archived by the owner on Nov 4, 2023. It is now read-only.

fix: fullsize popup cropped at the bottom #508

Merged
merged 1 commit into from
Oct 29, 2020

Conversation

akloeckner
Copy link
Contributor

fixes #423

this uses the first approach suggested in the issue.
I just couldn't get the other one to work...

fixes resoai#423

this uses the first approach suggested in the issue
I just couldn't get the other one to work...
@rchl
Copy link
Collaborator

rchl commented Oct 29, 2020

Seems fine to me.

In the future, we could probably just switch to flexbox which should make it easy to do without using vh units. flexbox has even better compatibility than vh unit as far as I can see on "caniuse".

@rchl rchl added the hacktoberfest-accepted Accept for Hacktoberfest label Oct 29, 2020
@rchl rchl changed the title fix: fullsize popup cropped at bottom fix: fullsize popup cropped at the bottom Oct 29, 2020
@rchl rchl merged commit 3cfaef8 into resoai:master Oct 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
hacktoberfest-accepted Accept for Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

History popup cropped on bottom
2 participants