Design comparison
Solution retrospective
I have no questions.
However, I am very open to criticism, so please recommend CSS best practices I should consider
Or identify any short-comings with my code or approach to this challenge.
Thanks in advance!
Community feedback
- @zalewskaewa7Posted 12 months ago
Hi! , congrats on completing this challenge πππ₯³π₯³
Here some tips to improve your solution. π―π
HTML:
Your main content should be conteined in the main tag. More info HERE
Header elements implements six levels h1-h6, with h1 being the most important and h6 being the least important. avoid omitting header levels, your page should start with the header h1 and successively with h2...h6. More info HERE
For responsive css you need using flexbox (more information here ) To make your page responsive, do the following:
.container {
width: 90%;
display: flex;
justify-content: center;
align-items: center;
max-width: 300px;
}
Then you don't need media queries in this project.
Your solution is very good, keep it up!
Happy coding π
1@eghosaclintonPosted 12 months ago@zalewskaewa7 π₯²yes I just saw there was no need for the media queries, thanks for the tip though!
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord