Insights!

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted over 3 years ago

  Hi

  It looks like you still need to make this work on mobile screens.

  Other feedback

  • those h2s definitely shouldn't be heading elements. List items would be more appropriate. Headings mean they have to be a heading for other content underneath them
  • never put font sizes in pixels, always rem/em (responsive unit)
  • use techniques like flex or css grid to create layouts, not large margins

  I hope this helps you

  0

  ---

  

  moon•60

  @moonbansalPosted over 3 years ago

  @grace-snow Hi! Yes I've recently learned responsiveness so I have to edit the code a bit, thank you for your input :)

  0