Skip to content

Feature/improve clear button hover #151

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
SouparnaChatterjee wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Feature/improve clear button hover #151

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
dfa039b
feat: improve Clear button hover and active states
SouparnaChatterjee Jan 7, 2026
891df74
feat: add glow effect on Clear button hover and active states
SouparnaChatterjee Jan 7, 2026
4db6ac5
Merge remote-tracking branch 'upstream/main' into feature/improve-cle…
SouparnaChatterjee Jan 14, 2026
9aa4c0b
feat: add matching glow effect to Filter button for UI consistency
SouparnaChatterjee Jan 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions components/Leaderboard/LeaderboardView.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -606,24 +606,26 @@ export default function LeaderboardView({
</div>

<div className="flex items-center gap-2">
{/* CLEAR BUTTON WITH GLOW EFFECT */}
{(selectedRoles.size > 0 || searchQuery || sortBy !== "points") && (
<Button
variant="ghost"
size="sm"
onClick={clearFilters}
className="h-9 hover:bg-[#50B78B]/20 cursor-pointer"
className="h-9 hover:bg-[#50B78B]/20 hover:text-[#50B78B] hover:shadow-[0_0_10px_rgba(80,183,139,0.3)] active:bg-[#50B78B]/30 active:text-[#50B78B] active:shadow-[0_0_15px_rgba(80,183,139,0.5)] transition-all duration-200 cursor-pointer"
>
<X className="h-4 w-4 mr-1" />
Clear
</Button>
)}

{/* UPDATED FILTER BUTTON WITH MATCHING GLOW EFFECT */}
<Popover open={popoverOpen} onOpenChange={setPopoverOpen}>
<PopoverTrigger asChild>
<Button
variant="outline"
size="sm"
className="h-9 border border-[#50B78B]/30 hover:bg-[#50B78B]/20 cursor-pointer"
className="h-9 border border-[#50B78B]/30 hover:bg-[#50B78B]/20 hover:text-[#50B78B] hover:shadow-[0_0_10px_rgba(80,183,139,0.3)] active:bg-[#50B78B]/30 active:shadow-[0_0_15px_rgba(80,183,139,0.5)] transition-all duration-200 cursor-pointer"
>
<Filter className="h-4 w-4 mr-1.5" />
Filter
Expand Down
2 changes: 1 addition & 1 deletion next-env.d.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
/// <reference types="next" />
/// <reference types="next/image-types/global" />
/// <reference path="./.next/types/routes.d.ts" />
import "./.next/dev/types/routes.d.ts";

// NOTE: This file should not be edited
// see https://nextjs.org/docs/app/api-reference/config/typescript for more information.
Comment on lines 1 to 6

@coderabbitai coderabbitai Bot Jan 14, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Do not manually edit next-env.d.ts — this is an auto-generated file.

The file explicitly states on lines 5-6 that it should not be edited. Additionally:

  1. Using import instead of a triple-slash reference (/// <reference path="..." />) converts this from an ambient declaration file to a module, which can break global type augmentation.
  2. This change appears unrelated to the PR objective of improving the Clear button hover states.

Revert this file to its auto-generated state. If Next.js regenerated it with different content, commit only what Next.js produces without manual modifications.

🤖 Prompt for AI Agents 
In `@next-env.d.ts` around lines 1 - 6, The file next-env.d.ts was accidentally
converted into a module by adding an import (import
"./.next/dev/types/routes.d.ts"), which breaks global type augmentation and
contradicts the auto-generated warning; revert this file to the auto-generated
state by removing the import and restoring the original triple-slash reference
(or whatever Next.js currently generates), e.g., replace the import with the
ambient reference pattern (/// <reference path="./.next/dev/types/routes.d.ts"
/>) or simply revert the file to the exact content Next.js outputs so it is not
a module.

2 changes: 1 addition & 1 deletion tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
"moduleResolution": "bundler",
"resolveJsonModule": true,
"isolatedModules": true,
"jsx": "preserve",
"jsx": "react-jsx",
"incremental": true,
"plugins": [
{
Expand Down