chore: update project delete button tooltip text#1464
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Greptile SummaryThis PR updates the tooltip and
Confidence Score: 5/5
|
| Filename | Overview |
|---|---|
| src/renderer/components/ProjectDeleteButton.tsx | Tooltip title attribute and TooltipContent text updated from "Delete project" to "Remove the project from Emdash"; the default aria-label prop value was not updated to match. |
Sequence Diagram
sequenceDiagram
actor User
participant Button as ProjectDeleteButton
participant Tooltip as TooltipContent
participant Dialog as AlertDialog
User->>Button: Hover
Button-->>Tooltip: Show "Remove the project from Emdash"
User->>Button: Click
Button->>Dialog: Open confirmation dialog
Dialog-->>User: "Delete project?" title + risks summary
User->>Dialog: Acknowledge risks (if any) + click Delete
Dialog->>Button: onConfirm()
Button-->>User: isDeleting spinner shown
Comments Outside Diff (1)
-
src/renderer/components/ProjectDeleteButton.tsx, line 39 (link)Default
aria-labelnot updated to match new tooltip textThe default value for the
aria-labelprop still reads'Delete project', while the visible tooltip andtitleattribute were updated to'Remove the project from Emdash'. If the component is ever rendered without an explicitaria-label(e.g. in a future call site or in tests), screen readers will announce a label that doesn't match the visible tooltip, creating an accessibility inconsistency.
Last reviewed commit: f8c9209
Summary