Desktop Version Done

Community feedback

• 

  Adam Turner•55

  @adamjamesturner93Posted over 4 years ago

  Source code looks good on the whole and is laid out nicely.

  • Nice use of variables and mixins in your scss

  Points to work on:

  • Try to avoid so many divs and use more semantic markup. Where you have <section class="content-container"> consider making this <main ...> then each <div class="content-container-items"> can become a <section>.

  • <li> elements must be child elements on <ul> tags.

  • Make sure to provide helpful 'alt' tags on images. These are vital for people using screen readers or on low bandwidth connections to get a sense of the image they are unable to see.

  1

  ---

  

  Shubham Verma•225

  @ShubhamVerma1811Posted over 4 years ago

  @adamjamesturner93 Thanks for your feedback. I'll work on those issues.

  0