3-column-preview cards

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted over 3 years ago

  Hey, great work on this one. Points for not using height: 100vh in here (really advocates not using it :>).

  The layout is really good both in desktop and mobile

  My suggestions would be that. Well this kind of personal preference right

  • Default border. It will be awesome if you remove the default border in the button elements. If you look at the button, there is a black lines surrounding it and that is the default. You could just border: none in the button selector.

  • Center. In your mobile state, your layout is onto the left side. It will be better(well for me) for you to make it centered. So adding margin: 0 auto in your boxOfboxes selector will be really awesome.

  Everything seems good. In terms of markup, I think you using too much element, like for the button, you nested it inside of a 2 div.

  But still, you did a really awesome work^^

  1
• 

  superOzzy•345

  @superozzyPosted over 3 years ago

  hey thanks for the advice,i updated your suggestions,it really looks better with the border:none;!Thank you very much : )

  0