four card feature section using HTML and CSS

Community feedback

• 

  Shashi Lo•1,345

  @shashiloPosted over 4 years ago

  Good job Shery. Everyone is on point here. The main idea here is paying closer attention to the details.

  • Download PixelPerfect and overlay the desktop-design.jpg on top of your project. You'll see where you are off from the design.
  • Check Font size, line-height, color, font-weight, etc.
  • Size of cards are too big. Need correct border-radius. Box-shadow needs to be toned down.
  • Center your project.

  1

  ---

  

  shrey dhawan•145

  @shrey1799Posted over 4 years ago

  @shashilo thanks!! that was very helpful.

  0
• 

  Rishi Purwar•285

  @rishipurwar1Posted over 4 years ago

  Great Job... I think you should increase border-radius by 1 or 2 and increase the border size... Overall It's looking good...

  1

  ---

  

  shrey dhawan•145

  @shrey1799Posted over 4 years ago

  @rishipurwar1 thanks!! agreed.

  0
• 

  Vasily•250

  @vasily-mishaninPosted over 4 years ago

  Nice code! IMHO title and subtitle text should be a little bigger

  1

  ---

  

  shrey dhawan•145

  @shrey1799Posted over 4 years ago

  @vasily-mishanin thanks !! yes i agree with you completely.

  0
• 

  TCassas•235

  @TCassasPosted over 4 years ago

  Small contribution, instead of using <div class="header"> you can use the <header> tag!

  1

  ---

  

  shrey dhawan•145

  @shrey1799Posted over 4 years ago

  @TCassas instead of using what ?? i guess you made a typo there!! thanks!!

  0