Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
Summary
Refactor the Files page and shared file preview experience with smaller, focused structural cleanup.
This change improves the Files page composition, sidebar behavior, loading states, preview layout, and file viewer consistency without changing API contracts or core business behavior. It also consolidates file preview scaling behavior across PDF, DOCX, sheet, and media previews, removes the old skeleton preview path, and keeps shared preview imports aligned after renaming the dialog module.
Change type
Affected areas
Verification
pnpm lintpnpm buildScreenshots, API examples, or logs
Not included. Changes are focused on Files page structure, loading behavior, preview layout, and shared preview component cleanup.
Configuration, migration, and compatibility notes
No configuration, database migration, public URL, CORS, API contract, or Swagger changes.
frontend/shared/components/file-preview/file-preview-dialog.tsxwas renamed topreview-dialog.tsx, and existing frontend imports were updated.Documentation
Security and privacy
Checklist
.pycfiles,.envfiles, and local storage data are not committed.