Skip to content

Revert "Pagination algorithm enhancements" - #5560

Merged
camertron merged 1 commit into
mainfrom
revert-5504-hectahertz/pagination-algorithm-enhancements
Jan 16, 2025
Merged

Revert "Pagination algorithm enhancements"#5560
camertron merged 1 commit into
mainfrom
revert-5504-hectahertz/pagination-algorithm-enhancements

Conversation

@camertron

Copy link
Copy Markdown
Contributor

Reverts #5504

We're seeing a11y-based dotcom test failures: https://github.com/github/github/actions/runs/12817716159/job/35741636983

@camertron
camertron requested review from a team as code ownersJanuary 16, 2025 21:53
@github-actions

Copy link
Copy Markdown
Contributor

👋 Hi, this pull request contains changes to the source code that github/github depends on. If you are GitHub staff, we recommend testing these changes with github/github using the integration workflow. Thanks!

@github-actionsgithub-actionsBot added the integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm label Jan 16, 2025
@github-actions
github-actionsBottemporarily deployed to storybook-preview-5560 January 16, 2025 21:56 Inactive
@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize
packages/react/dist/browser.esm.js104.18 KB (-0.03% 🔽)
packages/react/dist/browser.umd.js104.52 KB (+0.03% 🔺)

@camertron
camertron added this pull request to the merge queueJan 16, 2025
Merged via the queue into main with commit b8284ceJan 16, 2025
@camertron
camertron deleted the revert-5504-hectahertz/pagination-algorithm-enhancements branch January 16, 2025 22:09
francinelucca added a commit that referenced this pull request Jan 16, 2025
github-merge-queueBot pushed a commit that referenced this pull request Jan 30, 2025
* Revert "Revert "Pagination algorithm enhancements (#5504)" (#5560)"
This reverts commit b8284ce.
* Remove 'rel' attribute for disabled pagination
* Revert "Remove 'rel' attribute for disabled pagination"
This reverts commit 8c0c656.
* Set aria-disabled=true
* fix(Pagination): correct maxVisiblePages calculation
* test(Pagination): correct test
---------
Co-authored-by: Marie Lucca <40550942+francinelucca@users.noreply.github.com>
Co-authored-by: Marie Lucca <francinelucca@github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration-tests: recommendedThis change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@camertron@joshblack