Hexer• 3,680
@Phalcin
Posted
Great work overall man. Keep it up
Marked as helpful
1
Design comparison
Solution retrospective
I have completed the challenge easy bank landing page. Any improvements in my code you will like to suggest. Will be great a help in my learning. Thanks
Community feedback
Old• 6,220
@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
Always Use Semantic HTML instead of
divlike<main><header>, etc for more info -
Consider using the h1 element as a top-level heading only (all h1 elements are treated as top-level headings by many screen readers and other tools).
-
Images must have alternate text
if my solution has helped you do not forget to mark this as helpful!
Marked as helpful
1
Sourabh• 420
@SOURABH358
Posted
@Old1337 Thanks for your review, will keep that in mind.
1
Please log in to post a comment
Log in with GitHub
Join 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