[Issue]: tooltip copy (button-sim test) #40

New issue

Closed

opened 2026-06-19 08:31:27 +00:00 by shad0w · 1 comment

shad0w commented

2026-06-19 08:31:27 +00:00

Owner

Raised by

Test (button simulation)

Description

The sign-out button tooltip says 'Sign out'; change it to 'Log out'. Tiny cosmetic copy change.

Priority

### Raised by Test (button simulation) ### Description The sign-out button tooltip says 'Sign out'; change it to 'Log out'. Tiny cosmetic copy change. ### Priority P3

shad0w added the

portal

claude-queue

triaged

labels 2026-06-19 08:32:23 +00:00

shad0w commented

2026-06-19 08:32:23 +00:00

Author

Owner

Claude triage

Request: Change the sign-out button tooltip from Sign out to Log out. P3 cosmetic copy change.

Action items

Update the tooltip text on the sign-out button, which is the title attribute (the hover tooltip):
- App/components/layout/header.tsx:55 — title="Sign out" → title="Log out"
- App/components/layout/mobile-header.tsx:40 — title="Sign out" → title="Log out"
Update the e2e selector that depends on this title so the test suite stays green:
- App/tests/e2e/auth.spec.ts:47 — page.getByTitle("Sign out") → getByTitle("Log out")

Files / areas involved

App/components/layout/header.tsx (desktop header — has the title tooltip)
App/components/layout/mobile-header.tsx (mobile header — has the title tooltip)
App/components/layout/desktop-required.tsx:22 — has visible "Sign out" text (not a tooltip; see open question)
App/tests/e2e/auth.spec.ts — selects the button via getByTitle("Sign out")
App/tests/e2e/mobile/desktop-required.spec.ts — selects via role name /sign out/i (matches visible text, not the tooltip)

No DB, auth logic, permissions, payments, or external systems touched — only the displayed label string. The signOut() call behaviour is unchanged.

Open questions

Scope: tooltip only, or all "Sign out" copy? The issue explicitly says "tooltip", which is strictly the title attribute (two files). The visible label "Sign out" in desktop-required.tsx and the test name strings are separate. Recommendation: change the two tooltips as requested; optionally align desktop-required.tsx visible text + its /sign out/i selector for consistency, but that exceeds the literal request. An unattended run should do the literal tooltip change plus the dependent auth.spec.ts selector.

Routing rationale: Single-word copy change in two title attributes with one dependent e2e selector — localized, unambiguous, no DB/auth/payments/external impact, verifiable by type-check/lint/tests → claude-queue.

Routing: claude-queue

## Claude triage # Triage — Issue #40: tooltip copy (button-sim test) **Request:** Change the sign-out button tooltip from `Sign out` to `Log out`. P3 cosmetic copy change. ## Action items 1. **Update the tooltip text** on the sign-out button, which is the `title` attribute (the hover tooltip): - `App/components/layout/header.tsx:55` — `title="Sign out"` → `title="Log out"` - `App/components/layout/mobile-header.tsx:40` — `title="Sign out"` → `title="Log out"` 2. **Update the e2e selector that depends on this title** so the test suite stays green: - `App/tests/e2e/auth.spec.ts:47` — `page.getByTitle("Sign out")` → `getByTitle("Log out")` ## Files / areas involved - `App/components/layout/header.tsx` (desktop header — has the `title` tooltip) - `App/components/layout/mobile-header.tsx` (mobile header — has the `title` tooltip) - `App/components/layout/desktop-required.tsx:22` — has **visible** "Sign out" text (not a tooltip; see open question) - `App/tests/e2e/auth.spec.ts` — selects the button via `getByTitle("Sign out")` - `App/tests/e2e/mobile/desktop-required.spec.ts` — selects via role name `/sign out/i` (matches visible text, not the tooltip) No DB, auth logic, permissions, payments, or external systems touched — only the displayed label string. The `signOut()` call behaviour is unchanged. ## Open questions - **Scope: tooltip only, or all "Sign out" copy?** The issue explicitly says "tooltip", which is strictly the `title` attribute (two files). The visible label "Sign out" in `desktop-required.tsx` and the test name strings are separate. Recommendation: change the two tooltips as requested; optionally align `desktop-required.tsx` visible text + its `/sign out/i` selector for consistency, but that exceeds the literal request. An unattended run should do the literal tooltip change plus the dependent `auth.spec.ts` selector. Routing rationale: Single-word copy change in two `title` attributes with one dependent e2e selector — localized, unambiguous, no DB/auth/payments/external impact, verifiable by type-check/lint/tests → claude-queue. **Routing:** `claude-queue`

shad0w closed this issue

2026-06-19 08:32:39 +00:00