easybank landing page using Bootstrap Studio

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted over 2 years ago

  Hey,

  That's the thing about Bootstrap. It might be great for building things quickly (if you know what you're doing, that is) but not so great for challenges like this, when you need a highly customised CSS. It's just not made for it, so forcing Bootrap on this website will probably give you more headache than just doing it with vanilla CSS >.<

  Besides this, there's a few things you could look into:

  1. Your images are missing alt texts. An alt text must be always present, even for decorative images, in this case you can just leave them blank (adding an aria-hidden="true" as well can be helpful).

  2. Your navbar isn't contained in any landmark, wrapping it in a header would be nice.

  3. Your social media icons should be wrapped in links. Your footer links should also be links.

  4. Your CSS is a bit all over the place. Is there a reason you used inline styles for some and put other styles in the normal CSS file? Generally using inline CSS isn't the best idea when going with just vanilla html-css-js, I'd say.

  5. Using px for font-size is also problematic. If a user were to change their font-size in the browser, your website will not scale. You could use rem instead.

  Overall it's a good solution though on a pretty tough challenge, so well done ^^ If you fix some of the accessibility issues, it will be great!

  1

  ---

  

  Tabassum Fatima•215

  @tab21Posted over 2 years ago

  @FluffyKas hi

  thanks for going through it. And will definitely look into it.

  Well, it did give me more headaches by the way. :(.

  0