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 component main solution

#sass/scss
marvyn kiplagatβ€’ 50

@Marvyn21

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Rio Cantreβ€’ 9,710

@RioCantre

Posted

Hello there! Nice work with this one. Viewing your solution, I would suggest the following for you...

  • Add the background image in the body tag with the following properties...
background-position: top center;
background-size: cover;
background-repeat: no-repeat;
bacground-color: (add color value here);
  • In the .pay add color:white for the font and add background color for the hover state

Hope this helps and Keep it up!

Marked as helpful

0

marvyn kiplagatβ€’ 50

@Marvyn21

Posted

@RioCantre Thank you for your feedback.

1
Rio Cantreβ€’ 9,710

@RioCantre

Posted

@Marvyn21 No worries! I would appreciate it if you mark it as helpful. Cheers!

Marked as helpful

0
Anosha Ahmedβ€’ 9,340

@anoshaahmed

Posted

hey good job on this challenge! wrap everything in your body in <main> OR use semantic tags! if you're using div right after body, then give them a role=""; however, it's a little frowned upon to use role right after body, so try to stick with semantic tags instead. you can read more about landmarks here

here is a list i made of accessibility issues & best practices

hope this helps :))

Marked as helpful

0

marvyn kiplagatβ€’ 50

@Marvyn21

Posted

@anoshaahmed I appreciate the feedback.

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