Submitted almost 3 years ago

Mobile First Four Card Section using HTML and SASS

Fidel Lim• 2,775

@fidellim

Preview site View code

This is a solution for...

Four card feature section

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

Would love to hear your feedback! Thank you!

Community feedback

ApplePieGiraffe• 30,545

@ApplePieGiraffe

Posted almost 3 years ago

Greetings, Fidel Lim! 👋

It's nice to see you complete another challenge! 😀 Good work on this one! 👍 Your solution looks great and responds well! 👏

The only minor thing I suggest is using a single <h1> tag for the heading of the page (since it is really just a single heading, not two separate headings). You can easily style each part of the heading differently by wrapping one of them in a <span> tag. 😉

Keep coding (and happy coding, too)! 😁

Fidel Lim• 2,775

@fidellim

Posted almost 3 years ago

@ApplePieGiraffe Thankyou! yea true I could've done that too lol. Will definitely try to recall that if a similar case appears hahahah!

Eric Salvi• 1,330

@ericsalvi

Posted almost 3 years ago

Great job with this one. Not a whole lot to give feedback on. The only thing I noticed is that the cards do not have a white background. Maybe implementing that would be the only thing needed?

Some things you could try to implement for future solutions would be incorporating semantic HTML markup, https://www.w3schools.com/html/html5_semantic_elements.asp and the BEM naming convention?

Keep up the momentum!

Fidel Lim• 2,775

@fidellim

Posted almost 3 years ago

@ericsalvi thanks for the tip!

Patrick• 14,325

@palgramming

Posted almost 3 years ago

🌟🌟🌟🌟🌟 Looks Great... Very nice job!!

Fidel Lim• 2,775

@fidellim

Posted almost 3 years ago

@palgramming thank you!

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