Product preview card component

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 2 years ago

  Hey there, Catherine Is Online! 👋

  Great job on this challenge! 👏 Your solution looks good and responds well! 🙌

  One thing I'd suggest is to probably not set the font-size of the text in the body in px because we typically want to users to be able to change the size of all text on the site by the changing the root font-size of the document. In other words, setting font-size in em or rem wherever possible would be the better way to go in order to make your solution as accessible as possible. 😉

  Hope that helps. 😊

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  3

  ---

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted over 2 years ago

  @ApplePieGiraffe Thank for the feedback!

  1

  

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 2 years ago

  @catherineisonline

  You're welcome! 😊

  0
• 

  faizan•2,420

  @afaiz-spacePosted over 2 years ago

  Hey @catherineisonline, remove height: 450px; from main element.

  Marked as helpful

  1

  ---

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted over 2 years ago

  @afaiz-space Thanks!

  0
• 

  lilKriT•380

  @lilKriTPosted over 2 years ago

  Very cool and clean. It almost looks like the design is a screenshot of your implementation.

  Marked as helpful

  1

  ---

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted over 2 years ago

  @lilKriT Thank you

  0
• 

  DkP-Consult•60

  @DkP-ConsultPosted over 2 years ago

  Well good job ! Just may be use bold propertie for promo price :) Friendly, DkP

  Marked as helpful

  1

  ---

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted over 2 years ago

  @DkP-Consult Thanks

  0
• 

  Renan Tiscoski•1,240

  @DonUggioniPosted over 2 years ago

  Looks spot on! Great job 👍

  Marked as helpful

  1

  ---

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted over 2 years ago

  @DonUggioni Thanks!

  0
• 

  MazzCode•120

  @MazzGuillePosted over 2 years ago

  Hi, I think the only suggestion I can give you is to set the font size on HTML or body to 62.5%, this makes 1rem = 10px, that way you can use rem units to set specific font sizes on different elements. I.E., by setting font size to 62.5%, if I wanted a <p> tag to be 16px, I would set it like this:

  html { font-size: 62.5%;
  }

  .my-p-tag{ font-size: 1.6rem; }

  This also helps because you can use rem units for padding, margins or any other measurement you need

  Hope it's helpful for you!

  0

  ---

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted over 1 year ago

  @MazzGuille Thanks for the tips!

  1