@correlucas
Posted
πΎHello Szymon, congratulations for your solution!
You solution seems great, I saw that you've used mobile first to approach write your code. In overall your solution is okay. But look that your component changes to mobile too early and the card gets too big with the paragraph with a single line. My advice is that you set the media query
for another breakpoint or give the property max-width
to the paragraph to have all the text in the same line.
Try to use .class instead of id, id's are used for more specific things as forms and to recall Javascript, the class can get the work done in this smaller projects.
@media (min-width: 700px)
#container {
display: flex;
align-items: center;
justify-content: center;
min-height: 100vh;
max-width: 900px;
margin: 0 auto;
}
Hope it helps, congrats for this amazing solution! Happy coding!
Marked as helpful