This challenge was completed with sass and sass variables

Community feedback

• 

  Anosha Ahmed•9,300

  @anoshaahmedPosted almost 3 years ago

  hey good job on this challenge! your report above shows a few issues though:

  • wrap everything in your body in <main> OR use semantic tags! if you're using div right after body, then give them a role=""; however, it's a little frowned upon to use role right after body, so try to stick with semantic tags instead. you can read more about landmarks here
  • you should have at least one <h1> in your webpage

  here is a list i made of accessibility issues & best practices

  hope this helps :))

  Marked as helpful

  0
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  Hay ! Good Job you made it look nearly perfect to the preview

  These below mentioned tricks will help you remove any Accessibility Issues

  -> Add Main tag after body like it should be your container

  -> For 1st heading or h1 tag, use header tag and then inside the header put your h1 or h2 etc

  -> But use header tag only once in main heading element.

  Keep up the good work!

  Marked as helpful

  0
• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Nice job with this one. Looking at your solution, I would suggest the following for you...

  • Inside the .component adjust the width into 355px and add margin-top: 5rem;
  • It's common to use div to wrap the content but it's better to use a specific tag for semantic like main .Wrap the .component with main tag

  Overall you did well and Keep it going! Cheers!

  Marked as helpful

  0

  ---

  

  MajidiyM•20

  @MajidiyMPosted almost 3 years ago

  @RioCantre, thank you for your advice))

  1