React JS - Responsive Testimonials Slider

Community feedback

• 

  Abdul Khaliq 🚀•72,380

  @0xabdulkhaliq

  Posted about 2 years ago

  Hello there 👋. Congratulations on successfully completing the challenge! 🎉

  • I have other recommendations regarding your code that I believe will be of great interest to you.

  ---

  BODY MEASUREMENTS 📐:

  • The width: clamp(375px,100%,1440px) property for body is not necessary. because it's a block level element which will take the full width of the page by default.

  ---

  • So feel free to remove width: clamp(375px,100%,1440px) style rule from body this will help you to write efficient code and makes your code more reusable.

  ---

  .

  I hope you find this helpful 😄 Above all, the solution you submitted is great !

  Happy coding!

  00

  Marked as helpful
• 

  Kamlesh Rajesh Yadav•5,040

  @Kamlesh0007

  Posted about 2 years ago

  Congratulations on completing the challenge! That's a great achievement, and I'm sure you put a lot of effort into it. I really liked the way you approached the challenge and the code you wrote. You demonstrated a good understanding of the concepts and applied them effectively to solve the problem.I have a few suggestions to improve your code further. You need to add an alt attribute to the img tag because it provides a text alternative for the image, which is useful in Users with visual impairments rely on screen readers to access the content on web pages. The alt text helps the screen reader software to describe the content of the image, which makes the page more accessible. add below Lines of code

  <img src={`../images/${image}` }  alt="image-john">
  <img  src={`../images/${image}` } alt="image-tanya.">
  

  00

  Marked as helpful