fix banner card button did primum#249
Conversation
|
Someone is attempting to deploy a commit to the Bhuvansh's projects Team on Vercel. A member of the Team first needs to authorize it. |
BHUVANSH855
left a comment
There was a problem hiding this comment.
The updated button styling looks good visually, but there are a couple of issues that should be addressed before merging:
-
The
.whiteselector is too generic and may unintentionally affect other elements across the application. Please scope it to the banner section or use a more specific class name. -
The button has hover and active states but no
:focus/:focus-visiblestate for keyboard accessibility. Please add a visible focus style. -
Consider responsive adjustments for button padding on smaller screens to ensure a consistent mobile experience.
Please address these issues before merging.
📋 Pull Request — Improve Banner Button Styling
🔗 Related Issue
Closes #242
📝 Summary
Updated the banner action buttons to have a more premium and consistent appearance across the website.
✨ Changes Made
.whitebuttons with a modern and premium look.🎯 Result
The "View Products" and other banner buttons now blend seamlessly with the website design and provide a more polished, premium feel.
screenshot