All comments

• 

  Zhong Junjie•10

  @whxhlgySubmitted 4 days ago

  

  QR code component using css grid + flex

  1

  30
  

  Steven Nguyen•70

  @snguyen56Posted 1 day ago

  Nice job on the component!

  The component looks great, but I would suggest that you add some more semantic HTML to improve the code. Adding this can help the performance of your websites with search engines and improve accessibility.

  For example, you could add headings (h1-h6) to the start of sections instead of only using paragraphs. You can also add some landmarks (like header, main, or footer), which would help any screen readers parsing your site.

  Overall, good work!

  0
• 

  P

  Tom Winskell•130

  @tomwinskellSubmitted 1 day ago

  

  Article preview component #JavaScript #CSS #HTML

  2

  20
  

  Steven Nguyen•70

  @snguyen56Posted 1 day ago

  Component looks great!

  If you want to improve the user experience, try using a click toggle for the active state for the JS, instead of a mouse leave event.

  For more information on toggling classes in JS, refer to this link: https://www.w3schools.com/howto/howto_js_toggle_class.asp

  Overall, well done!

  Marked as helpful

  1