Design comparison
SolutionDesign
Solution retrospective
Hello everyone!
This is my first attempt at doing this. I only have two months learning HTML and CSS, and obviously this will have errors, so I hope you will tell me what you think about my development of this mini-project and what things could be changed and improved. It would be greatly appreciated!
Community feedback
- @brkclnPosted about 3 years ago
Hello there, Thanks for submitting your project. You did well, 3 - 4 thing i can say..
- avoid to set "height" not a good practice (most of time),
- I see unnecessary codes in the codes you wrote. Maybe you wanna look at,
- For the font size try to use "rem" instead of "em" [watch this] (https://www.youtube.com/watch?v=pautqDqa54I)
- Can add "main",
- Can add "container" better experince, good work, keep up :)
Marked as helpful1@SamCastle16Posted about 3 years ago@brkcln Thank you very much for your comment Burak, that helps me a lot to improve. I will consider your advice.
In the case of "heights", which components of the code could be removed?
Again, thanks a lot!
1
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