Feat/contributor button contrast#6094
Merged
JhaSourav07 merged 3 commits intoJun 20, 2026
Merged
Conversation
- Replaced blurry gradient border with clean focus glow ring - Replaced plain text 'Clear' with X icon button - Added icon container with subtle background - Styled counter as a pill badge - Refined spacing and proportions - Updated tests to match new UI
The global :focus-visible outline was overriding the input's own focus styling (cyan border + glow ring). Added focus-visible:outline-none to suppress the mismatched blue outline.
Contributor
|
@BistNaveenSingh is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
Contributor
📦 Next.js Bundle Size Report (Gzipped Sizes)
📊 Summary of Totals
|
github-actions
Bot
added
the
type:feature
Aamod-Dev
added
level:intermediate
Aamod-Dev
approved these changes
Jun 20, 2026
Aamod-Dev
left a comment
Aamod-Dev
left a comment
Collaborator
There was a problem hiding this comment.
Review Summary
Labels applied:
- level:intermediate — Touches 5 files with both component and styling changes. The search bar restructuring (ContributorsSearch.tsx:92-130), CSS layer addition (globals.css:382-390), and corresponding test updates require moderate attention to detail across multiple areas.
- quality:clean — Well-structured changes. The Clear button was properly converted from text to icon with �ria-label='Clear search' for accessibility. Test coverage was updated to match the new DOM structure. The @layer base wrapping in globals.css follows Tailwind best practices.
- type:feature — Enhanced contributor search bar with new visual design, glow effects, and pill-style counter.
- type:design — UI redesign of the search bar with improved contrast, icon container, and hover states.
- type:testing — Test files updated to use container.textContent assertions matching the new DOM structure.
Mentor: Aamod007
Assessment: Solid UI enhancement. The search bar is visually improved with better contrast, and the contributor count is more legible in pill format. Tests were properly adapted to the new structure.
Approved! 🚀
JhaSourav07
added
the
gssoc:approved
Contributor
|
🎉 Congratulations @BistNaveenSingh! Your PR has been successfully merged. 🚀 Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.
Keep building! 💻✨ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes #6050
Pillar
Visual Preview
Before :
After:
Checklist before requesting a review:
CONTRIBUTING.mdfile.localhost:3000/api/streak?user=YOUR_USERNAME).npm run formatandnpm run lintlocally and resolved all errors (CI will fail otherwise).feat(themes): ...,fix(calculate): ...).README.mdif I added a new theme or URL parameter.