Uh oh!
There was an error while loading. Please reload this page.
feat: add page navigator to bottom of the page - #2187
Conversation
## Summary This PR addresses issue OpenSignLabs#2162 by adding the page navigator at the bottom of the relevant page. ## Changes - Reused the existing page navigation / pagination component - Added the navigator at the bottom of the page - Kept the existing navigation logic unchanged - Improved usability by allowing users to move between pages without scrolling back to the top ## Related Issue ClosesOpenSignLabs#2162
@aryansondharva is attempting to deploy a commit to the prafull 's projects Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds Prev/Next page navigation controls to multiple PDF-related pages to improve document paging UX.
Changes:
- Import the shared
PrevNextcomponent into several pages. - Render
PrevNextunder the PDF viewer area with consistent centering/padding.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| apps/OpenSign/src/pages/TemplatePlaceholder.jsx | Adds PrevNext import and renders pager under the PDF area. |
| apps/OpenSign/src/pages/SignyourselfPdf.jsx | Adds PrevNext import and renders pager under the PDF area. |
| apps/OpenSign/src/pages/PlaceHolderSign.jsx | Adds PrevNext import and renders pager under the PDF area. |
| apps/OpenSign/src/pages/PdfRequestFiles.jsx | Adds PrevNext import and renders pager under the PDF area. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
aryansondharva
commented
Jun 30, 2026
Hi maintainers, this PR is ready for review. It adds the existing page navigator to the bottom of the PDF pages as requested in issue #2162. I'd appreciate a review when you have time. Thank you! |
Summary
This PR addresses issue #2162 by adding the page navigator at the bottom of the relevant page.
Changes
Related Issue
Closes#2162