[Issue]: Change sign-out tooltip to 'Log out' #24

New issue

Closed

opened 2026-06-18 22:23:10 +00:00 by shad0w · 2 comments

shad0w commented

2026-06-18 22:23:10 +00:00

Owner

Raised by

Triage pipeline test — via portal

Description

The sign-out button tooltip in the portal header says 'Sign out'. Please change it to read 'Log out' instead. Small cosmetic copy change.

Priority

P3 — Low

Context

Page: /dashboard

### Raised by Triage pipeline test — via portal ### Description The sign-out button tooltip in the portal header says 'Sign out'. Please change it to read 'Log out' instead. Small cosmetic copy change. ### Priority P3 — Low ### Context - Page: `/dashboard`

shad0w added the

portal

label 2026-06-18 22:23:10 +00:00

shad0w commented

2026-06-18 22:36:12 +00:00

Author

Owner

[Claude triage] Could not auto-triage this issue. A human should review it and add either claude-queue or interactive.

[Claude triage] Could not auto-triage this issue. A human should review it and add either `claude-queue` or `interactive`.

shad0w commented

2026-06-18 22:38:46 +00:00

Author

Owner

Claude triage

Summary

Pure cosmetic copy change: replace every occurrence of the string "Sign out" used in UI-facing
text (tooltip title attributes and visible button labels) with "Log out". No logic, schema,
auth, or permission changes are involved.

Action Items

App/components/layout/header.tsx line 55
- Change title="Sign out" â†’ title="Log out"
- This is the icon-only button in the desktop portal header; its title attribute is the
  browser tooltip that the issue explicitly references.
App/components/layout/mobile-header.tsx line 40
- Change title="Sign out" â†’ title="Log out"
- Same icon-only button in the mobile header; same pattern.
App/components/layout/desktop-required.tsx line 22
- Change visible button text Sign out â†’ Log out
- Not a tooltip, but the same label in the "Desktop Required" fullscreen overlay. Should be
  updated for consistency with the issue intent.
App/tests/e2e/auth.spec.ts line 47
- Change page.getByTitle("Sign out") â†’ page.getByTitle("Log out")
- Test queries by the title attribute; must match the new value.
App/tests/e2e/mobile/desktop-required.spec.ts â€” multiple lines (51, 52, 53, 96, 104â€“106, 109, 116, 119)
- Replace all /sign out/i regex patterns and "Sign out" string literals â†’ "Log out" /
  /log out/i
- These tests locate the DesktopRequired button by its visible text; must match the new label.

Files / Areas Involved

File	Change type
`App/components/layout/header.tsx`	`title` attribute (tooltip)
`App/components/layout/mobile-header.tsx`	`title` attribute (tooltip)
`App/components/layout/desktop-required.tsx`	Visible button text
`App/tests/e2e/auth.spec.ts`	E2E selector update
`App/tests/e2e/mobile/desktop-required.spec.ts`	E2E selector updates (multiple)

The JS-only file App/tests/e2e/dashboard/po-status-badges.js contains comments referencing
"Log out" / "sign-out" in a different flow (it navigates directly to /api/auth/signout rather
than clicking the button), so no change is needed there.

Open Questions

None. The copy change is fully specified ("Sign out" â†’ "Log out"), the affected surfaces are all
identified, and the E2E tests are the only verification artefacts that reference the old label.

Routing rationale: Fully localized, cosmetic copy-only change across three UI components and their
corresponding E2E test selectors â€” no schema, auth, permissions, or external-system impact; clear
acceptance criteria; verifiable by lint, type-check, and existing E2E suite after selector updates.

Routing: claude-queue

## Claude triage # Triage: Issue #24 â€” Change sign-out tooltip to 'Log out' ## Summary Pure cosmetic copy change: replace every occurrence of the string `"Sign out"` used in UI-facing text (tooltip `title` attributes and visible button labels) with `"Log out"`. No logic, schema, auth, or permission changes are involved. --- ## Action Items 1. **`App/components/layout/header.tsx` line 55** - Change `title="Sign out"` â†’ `title="Log out"` - This is the icon-only button in the desktop portal header; its `title` attribute is the browser tooltip that the issue explicitly references. 2. **`App/components/layout/mobile-header.tsx` line 40** - Change `title="Sign out"` â†’ `title="Log out"` - Same icon-only button in the mobile header; same pattern. 3. **`App/components/layout/desktop-required.tsx` line 22** - Change visible button text `Sign out` â†’ `Log out` - Not a tooltip, but the same label in the "Desktop Required" fullscreen overlay. Should be updated for consistency with the issue intent. 4. **`App/tests/e2e/auth.spec.ts` line 47** - Change `page.getByTitle("Sign out")` â†’ `page.getByTitle("Log out")` - Test queries by the `title` attribute; must match the new value. 5. **`App/tests/e2e/mobile/desktop-required.spec.ts` â€” multiple lines (51, 52, 53, 96, 104â€“106, 109, 116, 119)** - Replace all `/sign out/i` regex patterns and `"Sign out"` string literals â†’ `"Log out"` / `/log out/i` - These tests locate the DesktopRequired button by its visible text; must match the new label. --- ## Files / Areas Involved | File | Change type | |------|-------------| | `App/components/layout/header.tsx` | `title` attribute (tooltip) | | `App/components/layout/mobile-header.tsx` | `title` attribute (tooltip) | | `App/components/layout/desktop-required.tsx` | Visible button text | | `App/tests/e2e/auth.spec.ts` | E2E selector update | | `App/tests/e2e/mobile/desktop-required.spec.ts` | E2E selector updates (multiple) | The JS-only file `App/tests/e2e/dashboard/po-status-badges.js` contains comments referencing "Log out" / "sign-out" in a different flow (it navigates directly to `/api/auth/signout` rather than clicking the button), so no change is needed there. --- ## Open Questions None. The copy change is fully specified ("Sign out" â†’ "Log out"), the affected surfaces are all identified, and the E2E tests are the only verification artefacts that reference the old label. --- Routing rationale: Fully localized, cosmetic copy-only change across three UI components and their corresponding E2E test selectors â€” no schema, auth, permissions, or external-system impact; clear acceptance criteria; verifiable by lint, type-check, and existing E2E suite after selector updates. **Routing:** `claude-queue`

shad0w removed the

portal

label 2026-06-18 22:38:47 +00:00

shad0w added the

claude-queue

label 2026-06-18 22:40:41 +00:00

shad0w closed this issue

2026-06-18 22:43:07 +00:00

shad0w removed the

claude-queue

label 2026-06-18 23:00:17 +00:00

shad0w referenced this issue