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

First mobile and flex-box

@azyepes29

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 really felt better and more confidence with this my second challenge. Thanks for all comments in the first one and welcome to the new comments in this one.

Community feedback

darryncodes 6,430

@darryncodes

Posted

Hi Luis,

Awesome solution, really well done!\

You could clear down your accessibility/html report if you use some more descriptive elements. Swap <div class="main"> for <main class="main"> and <div class="attribution"> <footer class="attribution">. Semantic html is really important for accessibility and helps with seo.

All the best!

Marked as helpful

1

@azyepes29

Posted

@darryncodes Thanks, I'll apply it next time!

0

@Eric-Ferole

Posted

Well done Luis. It looks pretty much the same as the solution. Your HTML class are well written, but you could use BEM methodology to write your CSS.

Keep up !

Marked as helpful

1

@azyepes29

Posted

@Eric-Ferole Noted. Thanks Eric.

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