@jen067Submitted about 1 month ago
phucphan01866
@phucphan01866All comments
- @aysenurtatliSubmitted about 2 months ago
- @jolman009Submitted 2 months ago@phucphan01866Posted 2 months ago
Your is already so good :D! But for a little note, you forgot to center the cards for mobile screen. Hope this can help you :3.
0 - @tauanecustodioSubmitted 3 months ago
- @divyam4070Submitted 2 months ago@phucphan01866Posted 2 months ago
You forgot to make the card responsive to browser's srceen size, read about them at the first part of this path, also the %169.99 can be striked through using <del>$169.99</del>. Sorry for my bad English, hope this can help you.
0 - @Basheer-1Submitted 3 months ago@phucphan01866Posted 3 months ago
This is already nice, but i think your main card width is a bit too thin, try increase them to 50% screen witdh.
Marked as helpful0 - @alexmina12Submitted 3 months ago@phucphan01866Posted 3 months ago
This is already very good. I think you can improve this a bit by adding some transition to buttons's hover effect.
0 - @adrianalbinoSubmitted 3 months ago@phucphan01866Posted 3 months ago
This looks nice, but in the figma prototype they have effects as we hover on the card. You can check again and look for :hover selector on internet.
0 - @alekseibodeevSubmitted 3 months ago
