Skip to content

20 rows a page instead of 50 - #287

Merged
yorickdewid merged 1 commit into
mainfrom
tweak/page-size-20
Jul 27, 2026
Merged

20 rows a page instead of 50#287
yorickdewid merged 1 commit into
mainfrom
tweak/page-size-20

Conversation

@yorickdewid

Copy link
Copy Markdown
Contributor

Fifty rows is two screens of scrolling before you reach Volgende, on a list people work through by looking rather than by reading. Twenty is roughly what the viewport holds at a 34–38px row, so paging replaces the scrollbar instead of arriving after it.

  • PAGE_SIZE in services/explorer.ts: 50 → 20 (Rapportages).
  • RecoveryListView's local page size follows it. Same table, same job — two lists disagreeing about how long a page is is a detail you only notice by being wrong about it.
  • LANE_FETCH stays at 50. That is the Werkbank's lane ceiling behind a 50+ label, not a page; dropping it would turn honest counts into 20+ everywhere.

One knock-on worth knowing: the type filter narrows the visible page (GET /inquiry has no type parameter), so on a 20-row page it has less to work with. The popover already says so; this makes that caveat louder rather than newly untrue.

pnpm build green.

🤖 Generated with Claude Code

Fifty rows is two screens of scrolling before you reach Volgende, on a
list people work through by looking rather than by reading. Twenty is
about what the viewport holds at a 34-38px row, so paging replaces the
scrollbar instead of coming after it.

Herstel follows the same number: it is the same table doing the same job,
and the two lists disagreeing about how long a page is would be a detail
you notice only by being wrong about it.

Werkbank's LANE_FETCH stays at 50 -- that is a lane ceiling behind a "50+"
label, not a page.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@yorickdewid
yorickdewid merged commit 6406626 into main Jul 27, 2026
1 check passed
@yorickdewid
yorickdewid deleted the tweak/page-size-20 branch July 27, 2026 10:14
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