Responsive form

Community feedback

• 

  Mike Cheshire•410

  @mixchexPosted over 1 year ago

  Hi @LilithNixx,

  Currently, your preview site is flagged as dangerous. Not entirely sure why, but something to look into.

  In terms of your comments:

  • for scaling the background with the window, you can use the CSS property background-size:cover to achieve this.
  • For the "All page content should be contained by landmarks" comment, you should add certain HTML tags (<main>, <header> etc) or attributes (role="main") onto certain elements throughout your web page to help with accessibility. Find out more about these here.

  I hope this helps you on your journey in front-end development!

  Mike

  Marked as helpful

  1

  ---

  

  LilithNixx•190

  @LilithNixxPosted over 1 year ago

  @mixchex thanks for your time! I use a lot of divs to use as flexbox containers... Is there any role for that?

  0

  

  Mike Cheshire•410

  @mixchexPosted over 1 year ago

  @LilithNixx It's not too bad using <div> elements in this case, as long as you're using them efficiently. The roles are just to help with accessibility and help someone navigate the web page effectively. Consider them as signposts for what content is contained within an element.

  I hope that helps!

  1
• 

  LilithNixx•190

  @LilithNixxPosted over 1 year ago

  I added that of role and the message of "All page content should be contained by landmarks" keeps appearing, I don't understand what it means by landmarks.

  0