Skip to content

feat: add theme comparison page with dashboard previews#175

Merged
SamXop123 merged 4 commits into
SamXop123:mainfrom
vaishnavikhatri3:feat/theme-comparison-page
Jun 11, 2026
Merged

feat: add theme comparison page with dashboard previews#175
SamXop123 merged 4 commits into
SamXop123:mainfrom
vaishnavikhatri3:feat/theme-comparison-page

Conversation

@vaishnavikhatri3

Copy link
Copy Markdown
Contributor

Description

This PR adds a dedicated Theme Comparison page that allows users to preview and compare all available dashboard themes in one place. Users can now easily explore theme options without manually changing theme parameters and refreshing previews.

Related Issue

Closes #174

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that causes existing functionality to change)
  • Documentation update
  • Refactor
  • Performance improvement

Changes Made

  • Added a dedicated /theme-comparison route for theme exploration.
  • Created theme preview cards for all available dashboard themes using mock data.
  • Added a "View Theme Comparison" CTA below the Themes section on the landing page.
  • Implemented a responsive layout for desktop and mobile devices.
  • Improved theme discoverability and comparison experience.

Testing

  • Ran npm test — all tests pass
  • Tested manually (if applicable)

Manual Testing

  • Verified navigation from landing page to /theme-comparison.
  • Verified all theme previews render correctly.
  • Tested responsive behavior on different screen sizes.
  • Confirmed theme comparison page is accessible through the CTA button.

Screenshots

Recording.samdev-pulse.mp4

Screen recording attached demonstrating:

  • Landing page CTA
  • Theme Comparison page
  • Theme preview cards
  • Responsive behavior

Checklist

  • My code follows the project's code style
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings or errors
  • I have updated the documentation if necessary
  • My changes are scoped to a single issue

openhands-agent and others added 3 commits June 11, 2026 11:04
- Create /theme-comparison route with full dashboard previews for all 23 themes
- Add theme-comparison.route.js that renders mock data with each theme
- Create theme-comparison.html page with responsive grid layout
- Add 'View Theme Comparison' link below themes section on landing page
- Include copy-to-clipboard functionality for theme parameter codes
- Add corresponding styles for theme comparison CTA and page

Closes SamXop123#174
@vercel

vercel Bot commented Jun 11, 2026

Copy link
Copy Markdown

@vaishnavikhatri3 is attempting to deploy a commit to the Dot_NotSam's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vaishnavikhatri3

Copy link
Copy Markdown
Contributor Author

Hi @SamXop123 , I have made the requested changes. Kindly review them and let me know if anything else needs to be updated.

@SamXop123 SamXop123 left a comment

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.

Excellent work @vaishnavikhatri3
just a few things that need to be fixed

Please remove that above card as shown in image. and also move the theme comparison button below the "show all themes" button and please also remove that long description line "Compare all themes side-by-side with full dashboard previews"

image

@vaishnavikhatri3

Copy link
Copy Markdown
Contributor Author

Hi @SamXop123 , I have updated the placement. Is this okay, or should I move it below?
image

@SamXop123

Copy link
Copy Markdown
Owner

Hi @SamXop123 , I have updated the placement. Is this okay, or should I move it below?

I think it should do just fine now.

@vercel

vercel Bot commented Jun 11, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
samdev-pulse Ready Ready Preview, Comment Jun 11, 2026 5:45pm

@SamXop123 SamXop123 left a comment

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.

Great work! Thank you for your contribution!

@SamXop123 SamXop123 merged commit 3536367 into SamXop123:main Jun 11, 2026
4 checks passed
@github-actions

Copy link
Copy Markdown

🎉 Congratulations @vaishnavikhatri3! 🎉

Your pull request has been successfully merged into samdev-pulse! Thank you so much for your valuable contribution and effort. Every single improvement helps make samdev-pulse a better profile experience for everyone! 🙌

🚀 What's Next?

  • Keep the momentum going! Feel free to browse through our open Issues and grab another one that interests you!
  • Support the Project: If you enjoyed contributing to samdev-pulse, please take a second to star our repository! It helps increase the project's visibility and supports our community. ⭐

💬 Stay Connected:
Have any questions, want feedback, or just want to discuss further contributions? Feel free to reach out directly to the maintainer on Discord: dot_notsam!

Thank you again, and keep up the amazing work! 💻✨

@vaishnavikhatri3

Copy link
Copy Markdown
Contributor Author

Thank you for the review and for merging the PR!

I really enjoyed working on this issue. If you need any help with future features, improvements, or fixes, feel free to tag me. I'd be happy to contribute again.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Add Theme Preview Gallery to Landing Page

3 participants