Byron• 2,180
@byronbyron
Posted
Hi @Reandyx
Looks really good!
Few accessibility issues flagged the FE Mentor report:
- Heading levels should only increase by one
- Document should have one main landmark
Both should be fairly easy to fix by making sure your heading start at <h1>
and continue in order. The second issue can be fixed by wrapping all the content in a <main>
tag.
Your transitions when the theme switches are brilliant!
0
Reandyx• 310
@Reandyx
Posted
@byronbyron yeah i know i did good job though thanks for feedback!! :D
0