Skip to content

EVA-122: credential + program list UX improvements (D7, D8)#117

Merged
suzil merged 1 commit into
mainfrom
susannahjd/eva-122-credential-program-list-ux-improvements-d7-d8
Mar 2, 2026
Merged

EVA-122: credential + program list UX improvements (D7, D8)#117
suzil merged 1 commit into
mainfrom
susannahjd/eva-122-credential-program-list-ux-improvements-d7-d8

Conversation

@suzil
Copy link
Copy Markdown
Owner

@suzil suzil commented Mar 2, 2026

Summary

  • Add right-click context menu with Rename to program list items (reuses existing ContextMenu portal component)
  • Make credential delete button always visible — removes opacity-0 group-hover:opacity-100 so the delete icon is visible without hovering, and naturally keyboard-accessible via Tab
  • Add E2E test for right-click rename flow; use exact: true on the getByRole to avoid collision with the hover pencil button ("Rename program")

Test plan

  • New E2E test: renames a program via right-click context menu passes
  • All 13 program-lifecycle + credentials tests pass (npx playwright test program-lifecycle credentials)
  • Existing rename-via-pencil test still passes

Closes EVA-122

Made with Cursor

- Add right-click context menu with Rename to program list items
- Make credential delete button always visible (remove opacity-0 hover trick)
- Add E2E test for right-click rename flow; use exact: true to avoid
  collision with the hover pencil button whose accessible name is
  "Rename program"

Made-with: Cursor
@suzil suzil merged commit 8165b17 into main Mar 2, 2026
2 of 4 checks passed
@suzil suzil deleted the susannahjd/eva-122-credential-program-list-ux-improvements-d7-d8 branch March 2, 2026 01:02
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.

1 participant