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

Responsive landing page HTML and CSS

@emilunageo

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I would like feedback on my code. I think I can simplify css but I don't know how. Would be pretty nice if I can get an advice.

Community feedback

@mikedev12

Posted

Hey Emiliano, thanks for your recommendation regarding my solution. It's very much appreciated. You can reach me on my telegram @codewithmike or WhatsApp me on +233208960450 so we can learn together to improve ourselves. Thanks a lot mate.

Marked as helpful

1

@emilunageo

Posted

Sure, perfect! I'll being in touch. I already sent you a message :)

0

@mikedev12

Posted

Okay @emilunageo

0
argel omnes 1,800

@argelomnes

Posted

Hey Emiliano,

For simplifying CSS, my suggestions are:

  1. Keep your specificity low e.g. .container .card .content-card h1 could've been h1 or .content-card h1 or .card-heading.

  2. You can remove border-radius from the image and add overflow: hidden to .card. This is more reusable. e.g. if you make the card more rounded later on or remove it, you only change it in one place.

Marked as helpful

1

@emilunageo

Posted

Hey @argelomnes tnx! Going to apply this :)

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