All comments

• 

  folake• 230

  @thefolake

  Submitted over 2 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

  50

  

  Kenneth• 240

  @ken0063

  Posted over 2 years ago

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

  0
• 

  Amr Elsayed• 10

  @AmrSayed74

  Submitted over 2 years ago

  Please review my solution and give me feedback on it

  

  NFT preview card component

  2

  60

  

  Kenneth• 240

  @ken0063

  Posted over 2 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

  This comment has received 1 replyShow