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

Employed grid and flex

#bootstrap#django#jss#react#react-native

@raphaelnnadi

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 need your suggestions . Thank you

Community feedback

P

@BurritoDoggie

Posted

Hello there!

Amazing job here! You did really well on this challenge! One thing you could add here is a div on the image at the top. Then you can add border-radius to the top corners. Great effort!

Keep Coding!

:)

(If you found this helpful you can let me know by marking this helpful or by giving this a thumbs up!)

Marked as helpful

2

@raphaelnnadi

Posted

@BurritoDoggie thank you for you time. I had a difficult time trying to fit the image to the card so i could give it a border radius but still couldn't. So i just matched it's width with the card . i have tried wrapping the img in a div like you said but it's still not responding to the border radius styling.

Don't know what to do

1
P
Jessica 880

@perezjprz19

Posted

@blacp what if you gave a class to the image itself and gave border radius to that, rather than the image-wrap div?

Marked as helpful

1

@raphaelnnadi

Posted

@perezjprz19 wow, thank you so much . I did exactly like you said and the image now responded to the border radius.

Thank you so much

1
P
Jessica 880

@perezjprz19

Posted

@blacp you're welcome. I'm glad that I was able to help :-)

Marked as helpful

1
P
Jessica 880

@perezjprz19

Posted

Hey, you're almost there. A few things to work on and you should be a little closer to your goal.

If you wrap the div with the class .card inside <main> tags, that will get rid of your accessibility errors in the report.

The <div> with the class .container can be replaced by <button>. It should look something like this <button>Button Text goes here </button>. The reason for this is because divs have no semantic meaning. You can make them look like things and mimic the same actions in some cases, but they are still not the real thing.

Happy coding!!

Marked as helpful

1

@raphaelnnadi

Posted

@perezjprz19 followed your advise and implemented your corrections , really helpful.

Thank you so much for your time

1

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