Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added min-height 100vh #72

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added min-height 100vh #72

wants to merge 1 commit into from

Conversation

avanderhoek
Copy link
Contributor

No description provided.

@boudewijn-zicht
Copy link
Contributor

Ik kan dit niet beoordelen ben ik bang. Geen idee wat 100vh uberhoubt betekend ☺️

Copy link
Contributor

@7ochem 7ochem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ik denk dat het een zinvolle toevoeging kan zijn. Alleen is altijd de vraag of je het in elk project nodig gaat hebben. Je voegt wel weer 1 + aantal breakpoints aan regels CSS toe natuurlijk. Dat is niet veel, maar je wilt ZSS niet helemaal vol hebben staan met dingen die je misschien wel eens zou kunnen gebruiken in projecten. Dan is alles bij elkaar wel veel.

@avanderhoek
Copy link
Contributor Author

Heel goed punt Jochem! Ik werk zelf wel steeds meer met vw en vh als eenheid (viewport width en viewport height). Procenten zijn lang niet altijd handig om mee te werken. Dit is er 1 die ik mezelf (zolang we met zss nog blijven werken dan) wel in elk project zie kopieren. Omdat ik het nu weer in een project moest zetten dacht ik, geen woorden maar daden. Ipv dat ik m de volgende keer moet kopieren meteen in zss stoppen.

@avanderhoek
Copy link
Contributor Author

Ik zou als tegenprestatie iets kunnen deleten uit zss 🤭

@7ochem
Copy link
Contributor

7ochem commented Sep 10, 2021

Ik zou als tegenprestatie iets kunnen deleten uit zss

Hahaha 🤣 Ik gok dat je vast een hoop kunt verzinnen...

Is @Hangloozz het eens met het toevoegen van vh? Gebruik jij dit ook veel?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants