FlexBox

Community feedback

• 

  Jason Heys•175

  @heyitsganyPosted about 3 years ago

  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 MALIK•345

  @Shahab-MalikkPosted about 3 years ago

  @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