Design comparison
SolutionDesign
Solution retrospective
I tried my best.
Community feedback
- @DrMESAZIMPosted almost 3 years ago
Well done @Salman311291. would you like to consider adding this footer code and its syling code just before the </body>
<div class="attribution"> Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank" >Frontend Mentor</a >. Coded by <a href="">SALMAN SAJIB</a>. </div> .otherwise you did good jobMarked as helpful0 - @PhoenixDev22Posted almost 3 years ago
Hello @Salman311291, Your solution is really nice. I recommend : For any decorative images, each imgtag should have empty alt="" and aria-hidden="true" attributes to make all web assistive technologies such as screen reader ignore those images, and for class="profile-photo" images alt="” text shouldn’t be empty. Read more about decorative and informative images . I really hope this feedback helps. Keep up the good work
Marked as helpful0
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