Skip to content

fix(miners): leaderboard row hover in light mode#113

Merged
entrius merged 3 commits into
testfrom
fix/volume-label-leaderboard-hover
Jun 1, 2026
Merged

fix(miners): leaderboard row hover in light mode#113
entrius merged 3 commits into
testfrom
fix/volume-label-leaderboard-hover

Conversation

@anderdc
Copy link
Copy Markdown
Collaborator

@anderdc anderdc commented Jun 1, 2026

Summary

Fix the Miner Leaderboard rows not highlighting on hover in light mode. The old surface.elevated hover color matched the table container's surface.light (both #f4f6f8) in light mode, so the change was invisible; dark mode worked by coincidence. Replaced with a mode-aware translucent overlay that reads in both themes.

Testing

  • Verified in the running app (light mode, live data): leaderboard rows highlight on hover.
  • tsc --noEmit clean.

anderdc added 2 commits June 1, 2026 15:07
… in light mode

- Register echarts GraphicComponent and add a VOLUME label above the
  volume sub-chart, mirroring the EMA title on the price chart.
- Miner leaderboard rows now highlight on hover in light mode; the old
  surface.elevated hover matched the table container surface (both
  #f4f6f8), so use a mode-aware overlay instead.
@anderdc anderdc changed the title fix(dashboard,miners): VOLUME chart title + leaderboard hover in light mode fix(miners): leaderboard row hover in light mode Jun 1, 2026
…d hover

Bump light-mode action.hover from MUI's faint 4% default to 6% (derived
from the brand woodsmoke via alpha) so hover reads over white card
surfaces. Leaderboard now uses the action.hover token like every other
table instead of surface.elevated / a hardcoded rgba.
@entrius entrius merged commit 033e039 into test Jun 1, 2026
2 checks passed
@entrius entrius deleted the fix/volume-label-leaderboard-hover branch June 1, 2026 20:18
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.

2 participants