Product Preview Card

Community feedback

• 

  ortiz-antonio•270

  @ortiz-antonioPosted 3 months ago

  • In .display, you override the font-size with a px unit:

    font-size: $fs-32;
    font-size: 32px;
  

  • A CSS methodology like BEM could be beneficial.

  • The breakpoint for the desktop layout is too large, in my opinion. A flipped tablet could show the desktop layout in this design.

  0

  ---

  

  P

  Sven Notermans•160

  @Sven-27Posted 3 months ago

  @ortiz-antonio yess your right. I was testing something and forget to erase the second one.

  I do know bem and for larger project i am gonna use it.

  Breakpoint is according design. But i do see what you mean. I keep it in mind.

  Thanks for your feedback.

  0
• 

  Jay Khatri•4,250

  @khatri2002Posted 3 months ago

  The design is really awesome and perfectly developed as per the given design reference!

  Just one little suggestion:

  The button should have the cursor: pointer; property. While the button already has a hover effect, adding pointer cursor is important for the following reasons:

  1. Improved User Experience: It visually communicates to users that the button is clickable, aligning with standard web interaction patterns.
  2. Consistency: Most interactive elements (like links and buttons) use the pointer cursor, so this maintains a consistent design language across the interface.

  This minor tweak will make the interaction even smoother and more intuitive for users.

  Happy coding!

  0

  ---

  

  P

  Sven Notermans•160

  @Sven-27Posted 3 months ago

  @khatri2002 Your right normally i give the button cursor pointer. In this case i simply forgot.

  1
• 

  P

  Tsukimi•820

  @hikawiPosted 3 months ago

  Woah this is really smooth and well done.

  I think you perfected this, there's really nothing to speak of haha. I don't know what to say, and no one commenting might discourage you. Amazing work!

  0