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

FlexBox

Alice 245

@AliceMenzie

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


  • card moves around on background, best way to make it fixed?

Community feedback

Jason Heys 175

@heyitsgany

Posted

Looks good! You've followed the design well. Just a couple of things to work on. Firstly, you haven't implemented the hover effects to the buttons and anchor tag.

Secondly, to help with accessibility (for screen readers) change the container div element to a main element instead.

Also, you haven't implemented the responsive design for mobile. So, your design touches the sides of the viewport at resolutions lower than 375px.

As for your card moving around on your background. I'm not sure what you mean.

0

@Shahab-Malikk

Posted

@Alice implemented everything well. You have not added hover effects to buttons(Proceed Payment) also add hover effect for anchor tag(Change). And the last thing I cannot understand your question so Did you mean that when we change screen size card move?

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