miro/files
1
0
Fork 0
Code Issues 1 Pull requests 3 Projects Releases 11 Packages 1 Activity Actions

Keep file toolbar and table header fixed while scrolling #97

Merged
miro merged 1 commit from claude/sticky-toolbar into main 2026-10-13 18:42:28 +00:00
Conversation 0 Commits 1 Files changed 5 +17 -12
miro commented 2026-10-13 18:37:51 +00:00
Owner
Copy link

On directory pages the whole content area scrolled, so the toolbar (upload, new folder, bulk actions) and table header could scroll out of view. Now only the table scrolls.

  • AppLayout gets a contained option that pins the page to the viewport height; the directory layout uses it
  • DataTable lets the table region take the remaining height and scroll, with a sticky header; the toolbar stays put
  • Table accepts a containerClassName so its inner wrapper doesn't become the scroll container
  • Other pages and the public share page are unchanged
On directory pages the whole content area scrolled, so the toolbar (upload, new folder, bulk actions) and table header could scroll out of view. Now only the table scrolls. - `AppLayout` gets a `contained` option that pins the page to the viewport height; the directory layout uses it - `DataTable` lets the table region take the remaining height and scroll, with a sticky header; the toolbar stays put - `Table` accepts a `containerClassName` so its inner wrapper doesn't become the scroll container - Other pages and the public share page are unchanged
Keep the file toolbar and table header fixed while the table scrolls
All checks were successful
CI / test (pull_request) Successful in 33s
Details
CI / lint (pull_request) Successful in 34s
Details
CI / typecheck (pull_request) Successful in 41s
Details
CI / knip (pull_request) Successful in 30s
Details
CI / docker (arm64, docker-arm64) (pull_request) Successful in 22s
Details
CI / docker (amd64, docker-amd64) (pull_request) Successful in 19s
Details
CI / format (pull_request) Successful in 32s
Details
CI / manifest (pull_request) Has been skipped
Details
CI / deploy-test (pull_request) Has been skipped
Details
CI / build (pull_request) Successful in 47s
Details
E2E / playwright (pull_request) Successful in 2m46s
Details
262c4cee48
miro merged commit 51e2c24c8f into main 2026-10-13 18:42:28 +00:00
miro deleted branch claude/sticky-toolbar 2026-10-13 18:42:28 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
miro/files!97
No description provided.