All comments

• 

  P

  Enziu•80

  @DefinitelyNotPatrick

  Submitted 10 months ago

  

  BLOG PROJECT

  #sass/scss

  2

  10
  

  P

  Andrew Shea•70

  @ashea29

  Posted 10 months ago

  The border radius on the card is a little too large compared with the design, but other than that it looks great! Nice job! 🚀

  0
• 

  P

  Enziu•80

  @DefinitelyNotPatrick

  Submitted 10 months ago

  

  QR CODE SOLUTION

  #sass/scss#accessibility

  1

  10
  

  P

  Andrew Shea•70

  @ashea29

  Posted 10 months ago

  Great job on getting it so close to the design! A couple suggestions to improve accessibility: (1) using semantic HTML instead of divs, where possible; (2) since the h2 is the only heading tag it would be best just to make it an h1, because skipping heading levels is a don't-do in WCAG.

  1

  Marked as helpful