Flexbox, Responsive Web Design

Community feedback

• Account deleted

  Hi!, congrats on completing this challenge. 🥳🎊

  Here some tips to improve your code. 🚀

  Background images are for decorative purposes, they do not add value to your website. What you can do to fix it is the following, add the aria-hidden='true' attribute and set the alt attribute to empty, like so alt=''. You can also choose to use the background-image property, thus avoiding using the above.

  Example:

  <img src="images/pattern-background-desktop.svg" id="img1" alt="" aria-hidden='true' >
  <img src="images/pattern-background-mobile.svg" id="img2" alt="" aria-hidden='true' >
  

  I can see that you have a little reset for the default styles that the browser gives, that's ok. But you can improve it by using other resets like the following:

  • Normalize.css
  • Reset.css
  • My Custom CSS Reset of Josh Comeau

  Your solution is very good. 💯

  Happy coding! ✌️🙃

  Marked as helpful

  0