Omar Mohy• 670
@omarmohy98
Posted
Hi@@Drougnov, wonderfully done! The code is remarkably responsive. Below is my feedback on potential improvements!
1.) your <a> tags not have discernible text that will make an accessibility issue, you can use aria-label="" to solve this problem, for example <a href="#"><i class="fab fa-facebook-f"></i></a> will be <a href="#" aria-label="Facebook Account"><i class="fab fa-facebook-f"></i></a>.
If you found my feedback helpful, please mark this comment as helpful. Wonderfully done. Happy coding!
Marked as helpful
1
ROCKY BARUA• 1,090
@Drougnov
Posted
@omarmohy98 Thank you so much. I was worried about the accessibility issues cause I didn't know how to solve them. So thanks again. Going to fix them right now :)
1
