Skip to content

fix(ui): reveal information-row actions on hover - #524

Merged
astyfx merged 1 commit into
mainfrom
fix/memory-item-buttons-invisible
Sep 10, 2026
Merged

astyfx merged 1 commit into
mainfrom
fix/memory-item-buttons-invisible

Conversation

@astyfx

@astyfx astyfx commented Sep 10, 2026

Copy link
Copy Markdown
Member

Summary

Information panel row actions (memory, plans, linked PRs) now appear on hover. Hover and keyboard focus-within previously lived on separate StyleX objects that both wrote --info-row-action-opacity, so the later class dropped the hover value and the buttons stayed invisible until a click focused them.

Changes

Fixes

  • ui: reveal Information panel row actions on hover without requiring a click to focus them

Test Evidence

  • bun test tests/information-row-styles.test.ts — pass
  • bun run typecheck — pass
  • bun run test:e2e -- tests/e2e/information-row-actions.e2e.ts — pass (hover on the memory row body raises trail opacity from 0 to 1 without focusing the buttons)

Notes

The same row style is shared by memory, saved-plan, and linked-pull-request lists, so all three pick up the fix.

Keep hover and focus-within on one StyleX custom-property declaration so row actions appear without a click.

Co-authored-by: Cursor <cursoragent@cursor.com>
@astyfx
astyfx requested a review from a team as a code owner September 10, 2026 15:53
@astyfx
astyfx requested review from JongSikLim and schang124 and removed request for a team September 10, 2026 15:53
@upwind-code-us

upwind-code-us Bot commented Sep 10, 2026

Copy link
Copy Markdown

Upwind Upwind Code Scan - ✅ Passed

0 newly introduced vulnerabilities · 0 resolved · 57 total in this PR vs main

Total breakdown: 🔶 6 High | 🟡 48 Medium | 🟢 3 Low

View full analysis in Upwind Console

Scan completed in 18s

Scan history (1 scan)
Commit Scanned at New Resolved Net
f153d6a < 2026-09-10 15:53 UTC 0 0 0

Last scanned: f153d6a · 2026-09-10 15:53 UTC

@upwind-code-us

upwind-code-us Bot commented Sep 10, 2026

Copy link
Copy Markdown

Upwind Upwind IaC Scan - ✅ Passed

0 newly introduced misconfigurations · 0 resolved · 0 total in this PR vs main

View full analysis in Upwind Console →

Scan completed in 3s

Scan history (1 scan)
Commit Scanned at New Resolved Net
f153d6a < 2026-09-10 15:53 UTC 0 0 0

Last scanned: f153d6a · 2026-09-10 15:53 UTC

@astyfx
astyfx enabled auto-merge (squash) September 10, 2026 15:53

@heath-s heath-s left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved

@astyfx
astyfx merged commit 160e055 into main Sep 10, 2026
6 checks passed
@astyfx
astyfx deleted the fix/memory-item-buttons-invisible branch September 10, 2026 16:01
@astyfx astyfx mentioned this pull request Sep 15, 2026
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