Music subscription plans!!!

Community feedback

• 

  P

  Øystein Håberg•10,710

  @Islandstone89Posted 11 months ago

  HTML:

  • Every webpage needs a <main> that wraps all of the content, except for <header> and footer>. This is vital for accessibility, as it helps screen readers identify the "main" section of a page. Change .main into a <main>.

  • "Proceed to payment" would probably navigate you to the next page - to me it makes more sense as a link.

  CSS:

  • It's good practice to include a CSS Reset at the top.

  • font-size must never be in px. This is bad for accessibility, as it prevents the font size from scaling with the user's default setting in the browser. Use rem instead.

  • Set font-family on the body and remove it elsewhere - this is more efficient than setting it on each part.

  • Do not use fixed widths and heights, except for on the music icon. You rarely want to set fixed dimensions, especially heights.

  0