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

J 330

@littledragonshrimp

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


Questions!

  • I had some difficulty changing the "Change" button, in the scss/css file it wouldn't change.

Community feedback

@amalkarim

Posted

Hi Jordan,

May I ask, what do you mean by changing the "Change" button? What do you want to change? I think your solution is quite great.

One thing, though. In accessibility report, we can see one error that says:

Interactive controls must not be nested

<button type="button" class="change">
        <a href="#">Change</a>
        </button>

To remove that error, don't use <button> and <a> at the same time. Choose only <button> or <a>, like this <button type="button" class="change">Change</button> or this <a href='#'>Change</a>.

Maybe this step could solve your problem too. Give it a try.

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