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

Vanilla HTML, CSS Grid and Flexbox

P

@shrki416

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


Hi all,

All feedback welcome :) Thanks so much!

Community feedback

Ali 260

@AleksHNZ

Posted

Hi Ahmed! Well done.

-My only recomendation will be to use use "a" tag, not the "button" tag for the buttons. Because if you will create real sites(I hope you will), these buttons will have to redirect to another page and the tag "a" is best for it.

Marked as helpful

3

P

@shrki416

Posted

@AleksHNZ awesome, hadn't considered that. Will make that change :)

0
P
David Turner 4,100

@brodiewebdt

Posted

Nice job. This is well done. @AleksHNZ is right about the a tag. I give them a class of .btn and other classes for styling multiple buttons (btn-primary, btn-secondary, btn-ghost, etc.)

Marked as helpful

1

P

@shrki416

Posted

@brodiewebdt Thank you! This is a great idea. I'll make those changes.

0
P
David Turner 4,100

@brodiewebdt

Posted

Your welcome.

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