I tried my best.
@DrMESAZIM
Posted
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 helpful
@Salman-sajib
Posted
@DrMESAZIM Thank you.
@PhoenixDev22
Posted
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 helpful
@Salman-sajib
Posted
@PhoenixDev22 Thank you.
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
