Skip to content
OverviewSolutions
3
Comments
1
Candice
@candiuk

All comments

  • Tony•50
    @Snorri967Submitted about 2 years ago

    I struggled to begin with implementing the main background-color as i thought the Main element would auto resize the the max-width of the widest child and couldn't think why I had so much whitespace when I changed the background-color to white... Yeah my brain was running slow, so I put everything inside a container and messed with the margins which corrected that for me.

    Order Summary Card Challenge

    3
    Candice•50
    @candiukPosted about 2 years ago

    Hi, well done on completing the challenge.

    • Placing your 'Order Summary' as an H1 and 'Annual plan' as H2 will remove the accessibility warning issue you're receiving.

    • The 'Change' link should not need or have the <p> wrapping the text inside the <a>. Remove the <p> or swap them around.

    • To achieve the hover effects, add styling with :hover. eg: button:hover { add your style } Here's a link to MDN's docs that you may find helpful.

    Marked as helpful
    0