Skip to content
Submitted almost 3 years ago

Responsive landing page using html, scss, javascript

Sourabh•420
@SOURABH358
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Easybank landing page
  • HTML
  • CSS
  • JS
3intermediate

Design comparison

SolutionDesign

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

  • Hexer•3,660
    @EmmanuelHexerPosted almost 3 years ago

    Great work overall man. Keep it up

    Marked as helpful
    1
  • Account deleted

    Hello there! 👋

    Congratulations on finishing your challenge! 🎉

    I have some feedback on this solution:

    • Always Use Semantic HTML instead of div like <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
    @SOURABH358Posted almost 3 years ago

    @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