Responsive Artical Card Using React

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted 12 months ago

  Hello there 👋. Good job on completing the challenge !

  Your design looks visually appealing, and you've done a good job overall. However, I have a few suggestions to enhance both the accessibility and structure of your code:

  • While using onClick on a div is technically functional, it's not semantically correct. A div is a container element, and using it as a clickable item without a clear purpose can be confusing for users, especially those who rely on screen readers. For better accessibility, consider using a <button> element instead. Buttons are designed for interaction and are more accessible.

  • For social media icons, it's good practice to use links (<a>) instead of just images. This allows users with screen readers to understand that these images represent links to social media profiles.

  • You should use the cursor: pointer property to indicate that the element like a button or a link is clickable.

  I hope you find it useful! 😄

  Happy coding!

  0

  ---

  

  Surya•200

  @Surya-ThiruvengadamPosted 12 months ago

  Thanks for your suggestion

  1