@DylandeBruijn
Posted
Hi again @BrianMunizSilveira,
Another great solution, looks very clean! I like that you added the attribution in the card, it fits well. Did you run into any trouble or did you have any specific questions?
I have a couple of friendly suggestions after viewing your solution:
Good idea to add some padding
to the body
element. However because you have height: 100vh
on the body
this causes a problem and overflow issues. I suggest the following change.
body {
background: hsl(225, 100%, 94%) url(../images/pattern-background-desktop.svg) no-repeat;
background-size: contain;
min-height: 100vh;
display: flex;
align-items: center;
justify-content: center;
padding: 1rem;
}
So I removed the width: 100%
because the body
is a block level element with a default width: auto
so it already takes up the full width of it's parent. I changed the width: 100vh
into min-width: 100vh
so the body still scales in height with the content inside.
I would use a
tags for the buttons as they seem to navigate the user to a different page and an a
tag is generally used for that. A button
tag is generally used to do an action on the page itself.
I hope you find my feedback helpful and happy coding!
Marked as helpful
@BrianMunizSilveira
Posted
Hey again😅 @DylandeBruijn,
I'm glad you're giving me great suggestions, this is what I need most at the moment. In this challenge I got a lot of help from a developer who recorded a video explaining this project (I used it as a basis so that I wouldn't feel too lost in my first challenge here at Frontend Mentor), so it was more 'hands on'. I will adhere to your suggestions for the final project (today if possible) for sure and in my opinion I'm doing well 😅.
Thank you very much for your help, the world needs more people like you, keep it up!
@DylandeBruijn
Posted
Hi @BrianMunizSilveira,
Thank you for your kind words! In my opinion there is nothing wrong with watching a video to find some guidance about a project. Everyone learns in a different way and sometimes you just need a way to start. I'm glad to help out and you are making good progress.