Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(pagination): wrap PaginationItem in list and fix hydration error #2294

Closed

Conversation

frankdavidcorona
Copy link

@frankdavidcorona frankdavidcorona commented Jan 5, 2024

Each PaginationItem within an unordered list for improved accessibility and to adhere to proper HTML semantics, ensuring that individual pagination buttons are recognized as part of a list structure.

This solve this Hydration issue:

SCR-20240105-mszg

correctness

Nest each PaginationItem within an unordered list for improved
accessibility and to adhere to proper HTML semantics, ensuring that
individual pagination buttons are recognized as part of a list
structure.
Copy link

vercel bot commented Jan 5, 2024

@frankdavidcorona is attempting to deploy a commit to the shadcn-pro Team on Vercel.

A member of the Team first needs to authorize it.

@frankdavidcorona frankdavidcorona changed the title fix: wrap PaginationItem in list for semantic fix(pagination): wrap PaginationItem in list and avoid hydration error Jan 5, 2024
@frankdavidcorona frankdavidcorona changed the title fix(pagination): wrap PaginationItem in list and avoid hydration error fix(pagination): wrap PaginationItem in list and fix hydration error Jan 5, 2024
@shadcn
Copy link
Collaborator

shadcn commented Jan 18, 2024

@frankdavidcorona We fixed this in #2190. I'm going to close this one but feel free to reopen if you have any questions. Thanks for the PR. Appreciate it.

@shadcn shadcn closed this Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants