Skip to content

Conversation

@Samridhi-2007
Copy link
Contributor

@Samridhi-2007 Samridhi-2007 commented Sep 23, 2025

issue #220

Description:
This PR updates the Shop page UI to align with the overall theme and improve readability, spacing, and interactivity.

Changes Made:
Updated product card backgrounds to match the dark theme (soft gray/gradient).
Restyled price buttons with subtle accent colors for better contrast.
Added hover effects on product cards (scale-up, shadow, smooth transitions).
Applied rounded corners and soft shadows for a modern look.
Improved font hierarchy: bold brand names, lighter product details, highlighted prices.
Fixed inconsistent padding and spacing between cards for better alignment.

Impact:
UI is now visually consistent and interactive.
Improved readability and modern aesthetic.

Screenshot :
Screenshot (753)
Screenshot (754)
Screenshot (755)

If any changes and improvement needed let me know

@github-actions
Copy link

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@vercel
Copy link

vercel bot commented Sep 23, 2025

@Samridhi-2007 is attempting to deploy a commit to the agamjotsingh18's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Sep 24, 2025

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

Project Deployment Preview Comments Updated (UTC)
trendhora-api Error Error Sep 24, 2025 7:49am

@agamjotsingh18
Copy link
Owner

Fetch the recent code

@Samridhi-2007
Copy link
Contributor Author

@agamjotsingh18 i updated

@agamjotsingh18
Copy link
Owner

Still the error persists, npm run build

@Samridhi-2007
Copy link
Contributor Author

@agamjotsingh18 should i raise a PR again ?

@agamjotsingh18
Copy link
Owner

Yes sure

@Samridhi-2007
Copy link
Contributor Author

@agamjotsingh18 i will raise a combine pr for issue #220 and #221 because i started working for issue #221 , is that okay?

@agamjotsingh18
Copy link
Owner

No how will I assign labels for both pr in one pr

@Samridhi-2007
Copy link
Contributor Author

@agamjotsingh18 okay i will raise 2 PR

@Samridhi-2007
Copy link
Contributor Author

@agamjotsingh18 raised a PR kindly look over that , Thankyou

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.

2 participants