Repository navigation
fix: improve object browser responsiveness and configure batch size - #281
Merged
Merged
Conversation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
Description
The Object Browser mounts every loaded object row, which makes details and preview interactions sluggish in large listings. Render only the visible object rows and let users choose a smaller request batch size.
ListObjectsV2.MaxKeys.Type of Change
Testing
All local checks passed, including 811/811 tests and the production build/static export:
Browser verification used the real application with mocked S3/API responses:
max-keysvalues, reload persistence, sequential loading through 3,000 objects, and stale-response handling when switching batch sizes.Checklist
Related Issues
Closes #274
Screenshots (if applicable)
The before/after captures use the same mocked 1,000-object listing.
Additional states: batch selector, dark mode, empty bucket, and error
Mobile batch selector:
Dark mode with batch selector:
Empty bucket:
403 error:
Additional Notes
@tanstack/react-virtualdependency; no new dependencies.cxymds/issue-274-screenshotsbranch, outside this PR's code diff.AGENTS.md, as required by the repository guidance.