Skip to content

Conversation

@carsonfarmer
Copy link
Member

@carsonfarmer carsonfarmer commented Sep 28, 2025

Change previously discussed with Jovi

Fixes APP-453

Copilot AI review requested due to automatic review settings September 28, 2025 15:58
@vercel
Copy link
Contributor

vercel bot commented Sep 28, 2025

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

Project Deployment Preview Comments Updated (UTC)
comps Ready Ready Preview Comment Sep 29, 2025 10:00pm
comps-staging2 Ready Ready Preview Comment Sep 29, 2025 10:00pm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the styling of a leaderboard button to improve visibility and differentiation by changing its color scheme from white background to blue background.

  • Changed button background from white to blue-500 for better visibility
  • Updated hover states to maintain proper contrast and visual feedback

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 28, 2025

📊 Test Coverage Report

Package Lines Statements Functions Branches
apps/api 21.74% 21.74% 54.49% 71.31% (-0.09%)
apps/comps 0.00% 0.00% 43.80% 43.80%
packages/rewards 100.00% 100.00% 100.00% 100.00%
packages/staking-contracts 100.00% 100.00% 100.00% 100.00%

@stephhuynh18
Copy link
Contributor

@timothyylim not sure if this should go in or #1312

Copy link
Member Author

carsonfarmer commented Sep 30, 2025

They are almost identical, so probably doesn't matter. I'll stop picking these up so I don't trample your PRs! Having said that I personally like this one because it keeps the white border, but it's @stephhuynh18's call!

@stephhuynh18
Copy link
Contributor

i'll defer to @jovirecall #1312 or this one? You discussed this one with @carsonfarmer butttt you also approved @timothyylim's in the linear ticket.

Copy link
Member Author

Meh, just use the other one, it looks great!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants