Skip to content

Conversation

Tharanishwaran
Copy link
Member

Description

This PR fixes #6689

Notes for Reviewers

Before :

image

After :

image

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Tharanishwaran <your-email@example.com>
@l5io
Copy link
Contributor

l5io commented Aug 13, 2025

🚀 Preview for commit 769d791 at: https://689c5a238c5b2d1f5db52f1d--layer5.netlify.app

@vr-varad
Copy link
Contributor

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT

Add it as an agenda item to the meeting minutes, if you would :)

Copy link
Contributor

@vr-varad vr-varad left a comment

Choose a reason for hiding this comment

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

image image

Not Responsive @Tharanishwaran

@l5io
Copy link
Contributor

l5io commented Aug 21, 2025

🚀 Preview for commit cbf040c at: https://68a6daaa220d9193c469aed5--layer5.netlify.app

@Tharanishwaran
Copy link
Member Author

@vr-varad,Got it! I'll fix the mobile responsiveness issue and push the update soon.

@M-DEV-1
Copy link
Member

M-DEV-1 commented Aug 24, 2025

Hi @Tharanishwaran, do you have any updates on the requested changes?

@vr-varad
Copy link
Contributor

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT

Add it as an agenda item to the meeting minutes, if you would :)

@vr-varad
Copy link
Contributor

@Tharanishwaran Any updates on this?

@l5io
Copy link
Contributor

l5io commented Aug 28, 2025

🚀 Preview for commit 663995f at: https://68affa45d6b78e0b345e4050--layer5.netlify.app

Signed-off-by: Tharanishwaran <your-email@example.com>
@l5io
Copy link
Contributor

l5io commented Aug 30, 2025

🚀 Preview for commit 30d1164 at: https://68b269427223a631b329e67a--layer5.netlify.app

@Tharanishwaran
Copy link
Member Author

image

@vr-varad, I’ve fixed the mobile responsiveness. Could you check and let me know if this looks good?

@Tharanishwaran Any updates on this?

@l5io
Copy link
Contributor

l5io commented Aug 30, 2025

🚀 Preview for commit a67be99 at: https://68b2927a64b167864c9fe2b1--layer5.netlify.app

@vr-varad
Copy link
Contributor

The Checks are failing.

Copy link
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

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

@Tharanishwaran @vr-varad Instead of creating new Previous and Next buttons, could you use the existing button from the Sistent page? You can refer to this button component
for consistency in design and a cleaner look.
image

Copy link
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

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

@vr-varad @Tharanishwaran For the responsive mobile view, we don’t need numbered pages since only the previous and next pages are visible on the screen. Instead, showing just the Previous and Next buttons will provide a cleaner and more user-friendly experience

image

@Tharanishwaran
Copy link
Member Author

@Rajesh-Nagarajan-11 Thanks for the feedback! I'll use the Sistent button component and simplify mobile to just Previous/Next. Will update soon.

@Namanv0509
Copy link
Member

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT

Add it as an agenda item to the meeting minutes, if you would :)

1 similar comment
@Namanv0509
Copy link
Member

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT

Add it as an agenda item to the meeting minutes, if you would :)

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.

[UI Enhancement] Add Previous/Next buttons to pagination component on resources page
6 participants