Grid, Responsive, basic JS

Community feedback

• 

  Shivraj K•400

  @Shivraj-K09Posted over 1 year ago

  Based on the code, here is my feedback:

  • Great use of semantic HTML. You have used appropriate tags such as nav, ul, and picture which makes your code easier to read and understand.
  • Your CSS code is structured well and easy to read. You have used appropriate class names and have grouped related styles together.
  • Use of media queries for responsiveness is great. However, in some cases, you are relying on fixed values such as min-width: 600px which might not work well across different devices. It's a good idea to use relative values such as em or % to ensure consistency across devices.
  • Consider reducing the number of font sizes used in your code. Having too many font sizes can make your code harder to read and maintain. Try to group related font styles together and use CSS selectors to apply them to different elements.

  ---

  Overall, great job on your code and keep up the good work! Happy Coding 😉🫡

  Marked as helpful

  1