Product preview card component

Community feedback

• 

  P

  Ruben de Man•400

  @developer-rubenPosted 6 months ago

  Hi!

  Here are my tips:

  • Remove some of the whitespace
  • Adjust the font to the design
  • The button improves the accessibility of the component, but a bit too large for my taste ;)

  Best, Ruben

  0
• 

  Isaac-Victor Providence•100

  @Emoji123-sPosted 7 months ago

  Hi @iliasminas, congratulations on completing the project. Upon reviewing, your solution differs from the preview design, and here are some improvements I think could be made

  1. Optimizing the area holding the text describing the image. Reduce the height each element has to better match the preview.
  2. I noticed there is only one font used. Every style needed for the project is listed in the style-guide.md file in the project directory, so be sure to check it out.
  3. The container itself. Reduce the width property to make the entire container smaller, and try using either absolute positioning or flexbox to position the entire container properly.
  4. Your mobile view actually looks very good! Though you could avoid the scroll property with the overflow: hidden; style, then adjust each element to fit the screen without scrolling.

  Have fun! 😉

  0