Skip to content

style(sidebar): raise non-active hover to 65% accent#569

Merged
stickerdaniel merged 1 commit into
mainfrom
fix/sidebar-hover-strength
Jun 15, 2026
Merged

style(sidebar): raise non-active hover to 65% accent#569
stickerdaniel merged 1 commit into
mainfrom
fix/sidebar-hover-strength

Conversation

@stickerdaniel

Copy link
Copy Markdown
Owner

Follow-up to #568. The 50% non-active row hover read too faintly on near-white sidebars, so hover was hard to distinguish from the resting row. Raise it to 65% of the accent so the hover is clearly visible while still sitting below the full-accent active row, and bump the reveal-overlay token to the matching 65% mix so the shortcut overlay stays aligned with the row.

Regression guard: not warranted, single opacity-value tweak with no logic or data contract a test could pin without a brittle pixel snapshot.

The 50% non-active row hover read too faintly on near-white sidebars. Raise it
to 65% so hover is clearly visible while staying below the full-accent active
row, and match the reveal-overlay token.

Regression guard: not warranted, single opacity-value tweak with no logic or data contract a test could pin without a brittle pixel snapshot.
Copilot AI review requested due to automatic review settings June 15, 2026 12:25
@stickerdaniel stickerdaniel enabled auto-merge (squash) June 15, 2026 12:25
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copy link
Copy Markdown
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
saas-starter 171d1f5 Commit Preview URL

Branch Preview URL
Jun 15 2026, 12:26 PM

@stickerdaniel stickerdaniel merged commit 3497843 into main Jun 15, 2026
11 checks passed
@stickerdaniel stickerdaniel deleted the fix/sidebar-hover-strength branch June 15, 2026 12:32
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