Skip to content

Added preview for consistent user experience - #635

Open
pragya-manna wants to merge 1 commit into
Durgeshwar-AI:mainfrom
pragya-manna:feat/628
Open

Added preview for consistent user experience#635
pragya-manna wants to merge 1 commit into
Durgeshwar-AI:mainfrom
pragya-manna:feat/628

Conversation

@pragya-manna

@pragya-manna pragya-manna commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

🔀 Pull Request

📌 Issue Reference

Closes #628


📝 Summary

  • What issue does this fix?
    PdfDocx.tsx and PdfXlsx.tsx bypass ToolPageTemplate and call useFileUpload / FileUploadArea directly. They already receive previewUrl back from the hook, but never destructured it or passed it down to FileUploadArea — so the preview never rendered even though the machinery for it already existed.

  • What major changes were made?
    frontend/src/pages/PdfDocx.tsx , frontend/src/pages/PdfXlsx.tsx : destructure previewUrl from useFileUpload(), pass previewUrl={previewUrl} to .

No new components, no styling changes, no backend changes , reuses the existing preview logic already present in FileUploadArea.tsx and useFileUpload.ts.


📸 Screenshots

Screenshot 2026-08-07 221137 Screenshot 2026-08-07 221350

✅ Checklist

  • My code follows the project's coding conventions
  • I have tested all impacted features
  • I have updated or added necessary documentation

🔗 Related Issues / PRs

Issue #628


🏅 Open Source Program Participation

GSSoC 2026


💬 Additional Notes

n/a

@vercel

vercel Bot commented Aug 7, 2026

Copy link
Copy Markdown

@pragya-manna is attempting to deploy a commit to the Durgeshwar's projects Team on Vercel.

A member of the Team first needs to authorize it.

@pragya-manna

Copy link
Copy Markdown
Contributor Author

Hi @Durgeshwar-AI,
I’ve submitted the pull request. Please do check it when you get a chance. Thank you!

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.

[Feature] Add pdf preview for consistent user experience

1 participant