Responsive Result Summery Component

Community feedback

• 

  Bryan Li•3,530

  @Zy8712Posted about 1 year ago

  Your solution is very solid. Great work!!

  The only main thing I'd suggest you to do is the remove repeated lines of code that appear both insider and outside your media query. Such as display: flex that appears in your .container class inside and outside your media query. You only need it once outside your media query and it carries over.

  Hope this helps 👍

  Marked as helpful

  0

  ---

  

  acabrie•170

  @acabriePosted about 1 year ago

  Thank you!@Zy8712

  0
• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted about 1 year ago

  Hello there 👋. Good job on completing the challenge !

  I have some small suggestions about your code that might interest you.

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

  • For icons, you should still use the alt attribute, but it should be empty (alt=""). This indicates to screen readers and other assistive technologies that the icon is decorative and that they should skip over it

  I hope you find it useful! 😄 Above all, the solution you submitted is great!

  Happy coding!

  Marked as helpful

  0

  ---

  

  acabrie•170

  @acabriePosted about 1 year ago

  Thank you!@MelvinAguilar

  1