Skip to content
Submitted over 2 years ago

Responsive easybank landing page

MOHAMAD NAZIR MD NAZRI•180
@NAZIRwill29
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 would love to have feedback and comment from all of you

Community feedback

  • Fluffy Kas•7,715
    @FluffyKasPosted over 2 years ago

    Hey,

    It looks great on all screens, well done! I only noticed a few things:

    1. Your images are missing alt texts. All images must have them, even if you just leave them empty (in case they're decorative images).

    2. Instead of using divs for lists you could go with ul (in the nav and footer links, for example).

    3. All your sections should go inside a main, because at the moment they aren't contained by any landmarks. ^^

    4. Your social media links should have aria-labels describing where the links lead (as they're images only, without any text description).

    Marked as helpful
    1
    MOHAMAD NAZIR MD NAZRI•180
    @NAZIRwill29Posted over 2 years ago

    @FluffyKas Thank you for your feedback.

    0

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