Design comparison
Solution retrospective
feedback is appreciated
Community feedback
- @Ridwan10000Posted 3 months ago
@Rabelahmed Well done. There should be white borders around the avatars. Also rethink of your font-sizing and line-height
Marked as helpful0@RabelahmedPosted 3 months ago@Ridwan10000 thank you , I added borders to avatars i did not notice them in the design pictures and changed the font size of h2 to default and removed line height from it too
0@Ridwan10000Posted 3 months ago@Rabelahmed Good job. Also note that before using h1 You should not use h2 tag. This is the rule from official specification of HTML. You may want to check your HTML code here
0@RabelahmedPosted 3 months ago@Ridwan10000 I know I was going to put h1 before main and put display none on it but forgot about 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