Skip to content

feat(hub): paginate and filter prototype gallery - #35

Open
Copilot wants to merge 3 commits into
mainfrom
copilot/hub-pagination-filtering
Open

Copilot wants to merge 3 commits into
mainfrom
copilot/hub-pagination-filtering

Conversation

Copilot AI commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Why

The Hub loads every registered prototype into the index before searching. As repositories add more prototypes, each visit loads more records and Build data than a reviewer needs. Closes #33.

What changes

Reviewers browse up to 24 prototypes at a time with Previous and Next buttons that retain link behavior. Search runs across the selected repository rather than only the visible page. The searchable Author filter uses Primer Brand's single-selection ActionMenu, puts Me first, and filters by the person who first registered each prototype. Cards and the list show that same saved author; later commits do not change attribution.

Search, author, and page state stay in the URL for reloads and browser history. Changing a filter starts at the newest matching results. Requests keep the previous cards visible while loading or after a failure, with an explicit retry action. Card and Build loading is limited to the selected page.

The authenticated full-app browser flow remains unverified. These screenshots show synthetic local Storybook examples in light mode, not a signed-in deployment.

Paginated index, 1280x900. The fixture uses two representative cards to illustrate the page count and right-aligned Next button.

Prototype index with matching count, author filter, search, and Next button

Author menu, 320x260 close-up. Standard selection rows show Me checked first, followed by All authors and a named author.

Primer Brand author menu with Me selected, All authors, and Mona Lisa

Copilot AI max review requested due to automatic review settings October 19, 2026 18:43
Copilot AI requested review from a team and nshki as code owners October 19, 2026 18:43

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because there is no eligible user to bill.

An enterprise owner can allow the organization to be billed for reviews like this by enabling Copilot code review for users without a license.

Copilot AI added 2 commits October 19, 2026 19:03
Add repository-scoped server search, signed cursor pages, and a searchable Me-first registration-author filter. Bound card and Build hydration to each selected page and preserve filters in browser history.

Refs #33

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Replace the custom author popup with Primer Brand ActionMenu selection rows while retaining bounded search and Me-first ordering. Style page navigation as secondary buttons without changing link behavior.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI force-pushed the copilot/hub-pagination-filtering branch from 6b5ed01 to 370b350 Compare October 19, 2026 19:08
Consolidate insertion stability into the prototype and author paging scenarios, share cursor-scope checks, and rely on existing local-identity coverage. Combine overlapping frontend filter flows and remove repeated assertions without changing application code.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

Add bounded pagination and author filtering to the prototype index

3 participants