P
ApplePieGiraffeβ’ 30,545
@ApplePieGiraffe
Posted
Hey there, harshj11! π
Great job on this challenge! π Your solution looks good and responds nicely! π
A couple of suggestions I have are,
- Setting the
alt
text for the icons in the "Advanced Statistics" section of the page to be an empty string so that it will be ignored by screen readers. Thatβs because those elements aren't necessary or important to the content of the page and as a result don't need to be read by screen readers. That will help clear up some errors on your solution report. - Adding a
max-width
to the main wrapper or container element that holds the content of the page to prevent the design from stretching out to be too wide on extra-large desktop screens. - Turning the navigation links in the header and footer of the page into actual links by wrapping each of those items in a link tag. The social media icons in the footer page should be links, as well.
- Making sure to add a heading element inside every
section
tag that you use (see your solution report). If there is no visible heading in the design, you might need to add one anyway, but hide it visually with CSS, or choose a different element altogether. π
Hope you find these tips helpful. π
Keep coding (and happy coding, too)! π
Marked as helpful
1
harshj11β’ 50
@harshj11
Posted
hello @ApplePieGiraffe , thank you so much for your valuable suggestions, I would surely work on them :)
1