All comments

• 

  João•240

  @joaovxsantosSubmitted almost 2 years ago

  

  3 Column Preview Card

  2

  30
  

  Amr_Hesham•160

  @amooriheshamPosted almost 2 years ago

  Great work my friend

  Here's some things to edit i hope you found it useful for you. You need look back at the font-size it's need to be increased. you set the body height to 100vh which will limit your content to be set in this height which also will make a trouble in small screens so give the body min-height as 100vh to make it's content height adapt as the need of content to show it decent in small screens.

  I hope you found that useful keep cool stuff

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Sonya Castro•80

  @Sonya-cSubmitted almost 2 years ago

  

  Order summary component using tailwind

  1

  30
  

  Amr_Hesham•160

  @amooriheshamPosted almost 2 years ago

  Hello there Great work keep going

  1
• 

  EDOARDO PITACCIO•180

  @EdoPitoSubmitted almost 2 years ago

  My solution for the Social proof, any feedback is always welcome.

  

  Social Proof

  1

  30
  

  Amr_Hesham•160

  @amooriheshamPosted almost 2 years ago

  Great work buddy. Here's some tips to enhance your design. Give your top section some margin-bottom to make room to split between top and bottom. In bottom you can give your flex some gap to also separate between the 3 boxes. I recommend you to read more about align items using flexbox

  That's pretty much it Happy night

  1

  This comment has received 1 replyShow
• 

  Ahmed•80

  @AhmedAlkhSubmitted almost 2 years ago

  I was unable to make the image div and the information div the same size so there is a slight difference at the bottom where they should be flush. I would appreciate any help in fixing this issue along with any others that are spotted. Any feedback is appreciated, thanks in advance!

  Edit: Changes from feedback have been implemented.

  

  Product preview card component solution

  4

  30
  

  Amr_Hesham•160

  @amooriheshamPosted almost 2 years ago

  Great work my friend.

  Here's some notes to be know you design looks decent on desktop although in Mobile screen it's unusable. So my advice to get back and tweak the code to make it work in other screen and to read more about media queries. For your question about the image bottom area you can go with set the display of block Or give the image height 100% which will solve that issue too. Keep coding

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Sebastián•130

  @smavilpSubmitted almost 2 years ago

  

  Card using CSS Flexbox

  2

  20
  

  Amr_Hesham•160

  @amooriheshamPosted almost 2 years ago

  Great work my friend.

  Here's some tips which i hope you accept that from a newbie. You can give the div with .main some padding in bottom to make space between the footer. You but you bottom area in side a footer to solve the accessibility issue. And that's it keep the great work.

  Marked as helpful

  2