All comments

• 

  Fatima• 145

  @FatimahFarooq

  Submitted about 3 years ago

  Your feedback is much appreciated.

  

  Mobile first approach using html,css and js

  2

  10

  

  Fatima• 145

  @FatimahFarooq

  Posted about 3 years ago

  I really appreciate your detailed feedback.I cannot thank you enough.What you've mentioned earlier regarding the 1440 px not being a breakpoint rather it's the layout size,I would like you to explain this a bit,if it's no trouble. Regarding what you mentioned about setting box-sizing:border-box,you are right,I need to find out more about this as I'm seeing it being used in all tutorials. Using proper html tags and adding an alt are such basics,I can't belive to have missed those..lol,will pay more attention. l'll be mindful of the height and width property as well,what you pointed out was great. The error-message attribute (aria-*) is new to me, I'll definetely read more about it. I'll check out the link you've shared too.Thank you so much,your feedback really gave me a lot to check out and learn.Thanks again.

  0
• 

  dstriz09• 95

  @dstriz09

  Submitted over 3 years ago

  Pretty happy with this one! Any general feedback is always welcome. Thanks!

  

  Article Preview Component - The Basics

  1

  10

  

  Fatima• 145

  @FatimahFarooq

  Posted over 3 years ago

  Good job. You could also add/remove a (show/hide) class on the author div as well when share icon is clicked.

  0
• 

  Calvin.A.S• 120

  @calvin17

  Submitted over 3 years ago

  Hi!!! ✨🎉, I have completed this challenge, I would appreciate any comment about my solution.

  Thanks.😊😊

  

  FAQ accordion card React & SASS

  1

  10

  

  Fatima• 145

  @FatimahFarooq

  Posted over 3 years ago

  I really liked the animation that happens whenever a faq is hovered upon.Great Job!

  Marked as helpful

  0