News Web

Community feedback

• 

  Maxime•430

  @maxime927Posted almost 2 years ago

  Hi Dimorio, how are you? you did a cool I can give you some tips to improve your solution if you agree with that :

  • The breakpoint : don't let the design become the mobile display under 1440px it is way to early, we can appreciate the desktop design until landscape tablet ( 1024px )

  • The side-container : Let's use the css attribute .side-content:last-child { border-bottom: none; } to avoid the border for the last element of your listing.

  • be carefull about spacings, for exemple, in your box .hero-content you used display:flex and i agree totally with that, don't be afraid to use this again for your div .hero-mess

  for exemple :

  .hero-mess { display: flex; justify-content: space-between; flex-direction: column; }
  

  And think about put the CTA in a wrapper to not have the button in full width :)

  Keep going, that's great :) Hope that this few lines help you...

  Maxime

  1
• 

  dimorioo•60

  @DimorioPosted almost 2 years ago

  i changed it up a bit and made it my own news site

  0