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

Order Summary | challenge html and css

Dannyā€¢ 10

@Danny-for

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


Hey, guys !! this is my first developed challenge, i hope i managed to apply the correct concepts of HTML and CSS. Please, if I can improve anything, let me know!!! thankful .

Community feedback

P
Daveā€¢ 5,245

@dwhenson

Posted

Hey @Danny-for congrats on completing your first challenge! šŸ™Œ

I think it looks good, and you've made good use of flex to align things nicely.

Some small suggestions:

  1. Maybe try a couple of different options on the middle flex container to see what layout matches - I wondered if space-around might work? Not sure though!
  2. A neat trick to center components like this on the page it to set display: grid on the body, along with place-content: center; and min-height: 100vh; - this should help you with a few challenges

Keep up the good work! šŸ‘

Cheers šŸ‘‹

Dave

Marked as helpful

5

Dannyā€¢ 10

@Danny-for

Posted

ohh Thank you! @dwhenson, I will try to put into practice your suggestions/feedback on how to centralize the components.

success šŸ‘‹šŸ½ Thank you so much

  • Danny
0
hardyā€¢ 3,660

@hardy333

Posted

Hi, this is great result for the first time - good job.

Few suggestion:

  • Make button full width of its container so that it will be the same size as .container__plans.
  • use smaller font-size on paragraph text
  • Button hover state background-color is too gray, try to change it and add box-shadow also.

Marked as helpful

2

Dannyā€¢ 10

@Danny-for

Posted

hello @hardy333 ! , thanks so much for the feedback . I will try to improve it.

Thank you so much !

  • Danny
0
Carl Jonesā€¢ 200

@jones9411

Posted

HI @danny-for,

Good job on the solution, I've just done this one myself as well. The advice I was given was to use more semantic HTML tags like main, section, header, etc.

Also maybe use some more padding/margin to add a little bit more space between elements

Well done, Carl

Marked as helpful

2

Dannyā€¢ 10

@Danny-for

Posted

Hi @jones9411, thank you very much for the feedback I will try to put it into practice and congratulations for developing this challenge too.

Thank you so much !

-Danny

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