Skip to content
OverviewSolutions
21
Comments
2
Kenneth
@ken0063

All comments

  • folake•230
    @thefolakeSubmitted almost 3 years ago

    Please I need help with the mobile responsiveness. There's some sort of space I can't account for.

    Social-Proof-Section

    2
    Kenneth•240
    @ken0063Posted almost 3 years ago

    Great job! You forgot to add no-repeat to the second background image which is making it show multiple times.

    0
  • Kenneth•240
    @ken0063Posted almost 3 years ago

    Great job!. To fix your accessibility issues you can change

    <div class="container"> to <main>, change all <article> to <div> , change <div class="attribution"> to <footer> and changing <h2> to <h1>. Remember to change the names in your CSS also. To center the page content add {display: grid; place-items: center; width: 100%; height: 100vh;} You can also add width to the styles for main and give h1 transition and font size.
    Marked as helpful
    1