Mohammed Hassan• 240
@MrMohammedMath
Posted
Hello and congratulations about finishing your first challenge I have some suggestions that I thought it will be helpful.
- make sure to have at least one
<main>tag. you can use it instaead of div with class container Note: this will remove the "All page content should be contained by landmarks" warning - try to make
.card-containertake the full width and height usingwidth: 100%; min-height: 100vh.
Marked as helpful
0
justinrieman• 30
@justinrieman
Posted
@MrMohammedMath Thank you! I think I was confusing the desktop-preview.jpg with the desktop-design.jpg which is what made me want to change the width of .card-container.
0
Mohammed Hassan• 240
@MrMohammedMath
Posted
@justinrieman just focus on desktop-design.jpg Happy coding.
0
