Skip to content

feat(ui): enhance merge file card layout and readability(nsoc'26)#68

Open
SrashtiChauhan wants to merge 18 commits into
JhaSourav07:mainfrom
SrashtiChauhan:feat/file-metadata-v2
Open

feat(ui): enhance merge file card layout and readability(nsoc'26)#68
SrashtiChauhan wants to merge 18 commits into
JhaSourav07:mainfrom
SrashtiChauhan:feat/file-metadata-v2

Conversation

@SrashtiChauhan
Copy link
Copy Markdown
Contributor

Closes #45

NSOC'26

Hi @JhaSourav07,

I have worked on improving the UI of file cards in the Merge tool to enhance readability and overall user experience.

What I did

  • Improved spacing and layout inside each file card

  • Clearly separated:

    • File name
    • File size
    • Page count
  • Enhanced typography for better readability and hierarchy

  • Added subtle hover effects (border, shadow, scale) for better interaction feedback

  • Improved thumbnail container styling for better visual depth

  • Enhanced visibility of action buttons (preview & remove)


Why this change

Earlier, the file card UI felt slightly cramped and metadata was not clearly structured.
Now, the information is easier to scan, especially when multiple files are added, making the merge process more user-friendly.


Result

  • Cleaner and more modern UI
  • Better visual hierarchy
  • Easier file verification before merging

Screenshot

Screenshot from 2026-04-30 00-34-43

Notes

  • No logic changes were made
  • Only UI/UX improvements

Status

Tested locally and working as expected.

Please review and let me know if any changes are needed.
Thank you!

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 29, 2026

@SrashtiChauhan is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 29, 2026

👷 Deploy request for quickstpdf pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 33b475d

@SrashtiChauhan
Copy link
Copy Markdown
Contributor Author

hi @JhaSourav07 ,
kindly review my pr.
thank you!

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
quick-pdf Ready Ready Preview, Comment Apr 30, 2026 5:38pm

Comment thread quickpdf@0.0.0 Outdated
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this file about?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this file about?

Could you please clarify which file you are referring to? I want to make sure I address the exact issue properly.

Comment thread eslint Outdated
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this file about?

@SrashtiChauhan
Copy link
Copy Markdown
Contributor Author

Hi @JhaSourav07 ,
Those files were unintentionally generated locally and got included earlier. They are not part of the actual feature, and the current PR now reflects only the intended UI changes.

Please let me know if anything else needs to be updated. Thanks!

@SrashtiChauhan
Copy link
Copy Markdown
Contributor Author

hi @JhaSourav07 ,
I have

Hi @JhaSourav07 , Those files were unintentionally generated locally and got included earlier. They are not part of the actual feature, and the current PR now reflects only the intended UI changes.

Please let me know if anything else needs to be updated. Thanks!

hi @JhaSourav07 ,
Could you please review the pr.

@SrashtiChauhan
Copy link
Copy Markdown
Contributor Author

hi @JhaSourav07 ,
could you please review the pr as i have fix the issues that was made unintentionally due to some checks that were not passing.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to improve the Merge tool’s file card UI for better readability and interaction feedback, but it also includes substantial (and behavior-changing) updates to the Watermark page that go beyond the PR description and linked issue scope.

Changes:

  • Update Merge PDF card layout (spacing, hover effects, thumbnail container styling, clearer metadata hierarchy).
  • Refactor Watermark page flow (new “Apply Watermark” step, new error UI, preview URL lifecycle changes, removal of watermark style/position controls).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File Description
src/pages/Merge/Merge.jsx Enhances Merge file card UI styling and metadata presentation.
src/pages/Watermark/Watermark.jsx Significant Watermark UI/behavior refactor (processing, preview handling, error handling), not aligned with PR scope.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/pages/Watermark/Watermark.jsx
Comment thread src/pages/Watermark/Watermark.jsx
Comment thread src/pages/Watermark/Watermark.jsx
Comment thread src/pages/Watermark/Watermark.jsx
Comment thread src/pages/Watermark/Watermark.jsx
Comment thread src/pages/Watermark/Watermark.jsx
@SrashtiChauhan
Copy link
Copy Markdown
Contributor Author

SrashtiChauhan commented May 10, 2026

hi @JhaSourav07 ,
one of my pr was merged but you haven't update points and you also not given label.
Kindly update my points and add label.

Screenshot from 2026-05-10 15-14-51 Screenshot from 2026-05-10 14-34-39

Thank You
NSOC'26

@SrashtiChauhan
Copy link
Copy Markdown
Contributor Author

@JhaSourav07 , Could you tell me when my points update on leaderboard and dashboard?

@JhaSourav07
Copy link
Copy Markdown
Owner

@JhaSourav07 , Could you tell me when my points update on leaderboard and dashboard?

I think in around 2-3 hours

@SrashtiChauhan
Copy link
Copy Markdown
Contributor Author

@JhaSourav07 , Could you tell me when my points update on leaderboard and dashboard?

I think in around 2-3 hours

Thank you so much. I also want to ask you Shriii19 /FlowForge i have also contributed to them but they are not adding labels and updating my points on leaderboard and dashboard although i have commented him in each pr around 10 pr i have contributed but they have only closed the issues but not give any label and updation in points. What should I do?

@JhaSourav07
Copy link
Copy Markdown
Owner

@JhaSourav07 , Could you tell me when my points update on leaderboard and dashboard?

I think in around 2-3 hours

Thank you so much. I also want to ask you Shriii19 /FlowForge i have also contributed to them but they are not adding labels and updating my points on leaderboard and dashboard although i have commented him in each pr around 10 pr i have contributed but they have only closed the issues but not give any label and updation in points. What should I do?

Contact NSOC team for that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level2 5 pts - Intermediate NSOC'26

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(merge): improve file card UI and readability in merge tool

3 participants