Darkstar• 1,000
@DarkstarXDD
Posted
- For this project you only need one landmark element and that's the
<main>. You can change your<div.outer-container>to be<main.outer-container>. Check this reference. - Yes, this would have been a perfect challenge to be done using the grid layout. Would have been much easier too. But nothing wrong with using flexbox either.
- I see you have created the title as two separate headings, using a
<h1>and<h2>. But it is just one single heading with just two separate styles applied for two separate lines. This is very important because currently when a screen reader user browsers this page, the screen reader will read that one heading as two separate headings, which can be confusing. What you can do is use a single<h1>and then inside that use a<span>tag to select second line of the heading and style it differently.
0
