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

Picture with caption #14

Closed
vasilisvg opened this issue Feb 20, 2017 · 3 comments
Closed

Picture with caption #14

vasilisvg opened this issue Feb 20, 2017 · 3 comments

Comments

@vasilisvg
Copy link

Dit probleem is complexer: Het plaatje moet altijd helemaal zichtbaar zijn in elke viewport. Dus maximaal 100vh, en maximaal 100vw. Bovendien moet je ook altijd kunnen zien dat er een onderschrift is. En dit moet werken met een landscape en een portrait plaatje …
En kan je je nog herinneren hoeveel em breed een paragraaf maximaal mag zijn?
En hier is er iets mis met de aangeleverde HTML. Hier moet je dus wel iets aanpassen (-:

@DaveBitter
Copy link
Owner

De paragraaf heeft een max-width van 30em, dat is de bedoeling toch?

@vasilisvg
Copy link
Author

Copypaste-foutje. Die regel had ik weg moeten halen. 30em is inderdaad prima.

@DaveBitter
Copy link
Owner

Ah oke! :) Heb het probleem met een modal opgelost die verschijnt bij het klikken op het plaatje

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

No branches or pull requests

2 participants