Skip to content

feat(vara-eth-idea/frontend): smooth pagination#2449

Open
vraja-nayaka wants to merge 3 commits intomainfrom
vt-feat-vara-eth-frontend-smooth-pagination
Open

feat(vara-eth-idea/frontend): smooth pagination#2449
vraja-nayaka wants to merge 3 commits intomainfrom
vt-feat-vara-eth-frontend-smooth-pagination

Conversation

@vraja-nayaka
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the table and pagination systems to support smoother view transitions and advanced caching, introducing the useTablePagination and usePaginationEffects hooks. It also migrates the activity panel state to a global atom and adopts queryOptions for better React Query integration. Feedback focuses on conditionally rendering sort icons for non-sortable columns, parameterizing page sizes in hooks for better reusability, ensuring page size multiples for correct placeholder mapping, and refining data slicing in the pagination utility.

Comment thread idea/vara-eth/frontend/src/components/ui/table/table.tsx
Comment thread idea/vara-eth/frontend/src/shared/hooks/use-table-pagination.ts Outdated
Comment thread idea/vara-eth/frontend/src/pages/code/code.tsx Outdated
Comment thread idea/vara-eth/frontend/src/shared/utils/pagination.ts Outdated
@vraja-nayaka vraja-nayaka requested a review from osipov-mit April 24, 2026 07:54
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.

1 participant