search

steven-tey / precedent

An opinionated collection of components, hooks, and utilities for your Next.js project.
https://precedent.dev
MIT License
4.69k stars 409 forks source link

Two <main> elements in the markup #21

Closed Aesop7 closed 1 year ago

Aesop7 commented 1 year ago

Hi, I noticed that there are two main elements in the markup. https://validator.w3.org/nu/?doc=https%3A%2F%2Fprecedent.dev%2F I'm sure it's not a big issue, but I'd hate for it to break a screen reader for someone.

steven-tey commented 1 year ago

oh great catch! do you wanna PR to fix this? AFAIK you can just change this line to <div> and it should work

steven-tey commented 1 year ago

Will be fixed by https://github.com/steven-tey/precedent/pull/22