Design comparison
Solution retrospective
Hello guys, I just completed this challenge. If you have any suggestions, I would really appreciate it. Thank youu
Community feedback
- @ToNisamBioJaPosted over 3 years ago
Good job on this one.
It is responsive and looks like the original design.
One thing I would suggest is that you use more class names. It isnt good practice to nest elements in css this much (for example: main header h1 span ). Try to look into some naming conventions to make the code a little tidier.
Overall great job. Keep it up and happy codding!
Marked as helpful0@blurrypxlPosted over 3 years ago@ToNisamBioJa Hi, thank you so much for your class naming suggestions, I really appreciate that :)
0 - @MojtabaMosaviPosted over 3 years ago
Hi, nice job, a small tips for writing better markup, try to make your class names as descriptive as you can, for instance c1 does not tell us about that it is but in contrast image-box does.
Keep coding :)
Marked as helpful0@blurrypxlPosted over 3 years ago@MojtabaMosavi Thanks for the suggestion, I'll apply it in the next challenge :)
0 - @palgrammingPosted over 3 years ago
⭐⭐⭐⭐⭐ Very nice job on this challenge!!
1@blurrypxlPosted over 3 years ago@palgramming Thank you, I appreciated it :)
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