responsive landing page with html and css

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hi, Anthony! 👋

  I am not sure why you are splitting the stylesheets into five different files. From my point of view as someone who wants to give feedback, it makes it harder to give feedback to your CSS. So, unless you have good reasons for doing that, I recommend merging all the CSS files into one CSS file.

  Not only it's going to make your site load faster (because it only needs to download and parse one CSS file) but also, makes it easier for people who want to see and give feedback on the CSS code.

  Some feedback.

  • There's no need to wrap all the img tags with figure element. The only reason to use figure is if you need to include a figcaption. Otherwise img tag is fine or uses div instead.
  • Those star icons are decorative images. They don't give meaningful information. So, I recommend leaving the alt="" empty.
  • Alternative text for images should not contain any words that are related to image. The semantic meaning of the img tag is already enough.
  • The text-reviews shouldn't be h3. I recommend using p instead.
  • Move the footer outside of the main element. footer inside the main would have no meaning (no corresponding role).

  Hope this helps. Happy coding! 😄

  Marked as helpful

  0