Mike Cheshire• 390
@mixchex
Posted
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:coverto 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
@LilithNixx
Posted
@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• 390
@mixchex
Posted
@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
