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

CSS and HTML

Exileurt 100

@Exileurt

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 thinks the code is "ok".. Any tips ?

Community feedback

@DaanWarming

Posted

I would research some more about HTML. Your div with the class title should be a <h1> tag for example. Text like "Annual Plan" and "$59.99/year" should be wrapped by a <p></p> tag and not a div.

The styling looks good! If you add some styling to the :hover state of the <a> tags it would have been perfect

0
Reuben👾 560

@Rubylenshy

Posted

Hi, @Exiluert 👋. Congratulations on completing this challenge, you did a nice job. You could got one suggestion:

  • The <div class="card"></div> should be wrapped in a <main></main> tag, for easy recognition by any web browser, it stands out as the main content of the web page or just rename the div to main.

Great Job once again @Exiluert

Happy Coding

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