Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Four Card Feature Section Master | HTML & CSS

Stephanie 100

@Sneflenie

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I would appreciate any tips or feedback :)

Community feedback

@kbrandon19

Posted

Hey @Sneflenie, your code is pretty clean which is a great thing but the only feedback I would offer is to be careful with naming classes. You have html .cards then html .card within it and if not careful can cause a problem, especially if there's 100s of lines of code. Try using names that describe more of what it could be like card-container, card-wrapper, etc. But try not to ponder on class names though haha, I have that problem sometimes, keep it simple. 👍🏾

Marked as helpful

1

Stephanie 100

@Sneflenie

Posted

@kbrandon19 Thank you! I will definitely keep that in mind 🙂

0
ChristBM 290

@ChristBM

Posted

Good job @Sneflenie I checked your code and it's ok. If you want to improve it, read more about semantic HTML tags and CSS BEM class notation. Semantic HTML tags BEM

Marked as helpful

1

Stephanie 100

@Sneflenie

Posted

@ChristBM thanks! I’ll check it out

0

Please log in to post a comment

Log in with GitHub
Discord logo

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