@visualdenniss
Posted
Hey there Jan,
nice work, but there seems to be few issues on desktop view.
It seems you have used: height: 75vh; but this is causing the container to fall short off its content. I'd suggest avoid giving fixed height or max-heights for elements/containers that contain text elements. Not only it might breakout your layout like in this case, but it will cause accessibility issue, when user changes their default font-size, text will overflow and break the layout, instead use paddings inside the container if necessary. It is better to use min-height etc. but never a fixed height. Also i'd recommend having min-height: 100vh for the body instead of height: 100vh.
Hope you find this feedback helpful!
Marked as helpful
@jkop22
Posted
@visualdenniss Thanks! I updated my solution and its much better now :)
@visualdenniss
Posted
@jkop22 Hey, happy to be helpful! Yes it is looking much better now! :)
i'd also suggest adding some margin on top and bottom of the card or padding to the body so the card does not touch the edges, which is the case for me on notebook.
