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

Simple card using html and css

Omarβ€’ 160

@omarsaleh1

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


how to make the project responsive on mobile

Community feedback

Abdulβ€’ 8,540

@Samadeen

Posted

Salam Omar!.. Cheers on completing the challenge.. I'd suggest you scale up your container a little then you'd be good to go.. Over all you did well. Secondly.. You might want to name your div without using the dash(-).. underscore(_) is better

Happy Coding!!!!

Marked as helpful

0
Oldβ€’ 6,220

@Old1337

Posted

Hello there! πŸ‘‹

Congratulations on finishing your challenge! πŸŽ‰

I have some feedback on this solution:

  • Always Use Semantic HTML instead of div like <main> <header> <footer> , etc for more info

  • headings should only increase by one use h1 first then h2 etc..

  • you can make it responsive by decreasing the container width on small screens using @media queries

i hope this is helpful and goodluck

Marked as helpful

0

Omarβ€’ 160

@omarsaleh1

Posted

@Old1337 Thanks, man!

0
Naveen Gumasteβ€’ 10,480

@NaveenGumaste

Posted

Hello omar ! Congo πŸ‘ on completing this challenge

Let's look at some of your issues, shall we:

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