Submitted almost 4 years ago

four card feature section using HTML and CSS

shrey dhawan• 145

@shrey1799

Preview site View code

This is a solution for...

Four card feature section

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

im a newbie to HTML and CSS and i would really appreciate any comments on the layout, overall appearance or how could i have made my code better.

thank you !!😊

Community feedback

Shashi Lo• 1,345

@shashilo

Posted almost 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.

shrey dhawan• 145

@shrey1799

Posted almost 4 years ago

@shashilo thanks!! that was very helpful.

Rishi Purwar• 285

@rishipurwar1

Posted almost 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...

shrey dhawan• 145

@shrey1799

Posted almost 4 years ago

@rishipurwar1 thanks!! agreed.

Vasily• 250

@vasily-mishanin

Posted almost 4 years ago

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

shrey dhawan• 145

@shrey1799

Posted almost 4 years ago

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

TCassas• 235

@TCassas

Posted almost 4 years ago

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

shrey dhawan• 145

@shrey1799

Posted almost 4 years ago

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

Please log in to post a comment

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord