TT-7431 speaker name chosen mobile

[sarahentzel]

No description provided.

[Copilot]

🟢 Approval recommended

The functional change is small and well-scoped, and the added test provides a regression guard (only minor test cleanup nits noted).

Pull request overview

This PR removes the mobile-specific selection handler from the SpeakerName dialog so speaker selection/add flows use a single code path across form factors, and adds a regression test to ensure the upload speaker field displays the chosen existing speaker name (rights-gated).

Changes:

• Removed useMobile branching and the separate mobile selection handler in SpeakerName, routing all selections through the unified handleSelectAndClose.
• Clarified the intent of handleSelectAndClose with updated inline comments.
• Added a new component-level test covering the “upload speaker field” rights-gated display behavior after selecting an existing speaker.

File summaries

File	Description
src/renderer/src/components/SpeakerName.tsx	Removes mobile-specific selection logic and uses one selection handler for all views.
src/renderer/src/components/SpeakerName.test.tsx	Adds a regression test that selects an existing speaker and asserts the chosen name is shown on the button.

Review details

Suppressed comments (1)

src/renderer/src/components/SpeakerName.test.tsx:76

• The describe block name still says “(mobile)”, but the component no longer branches on mobile/desktop behavior. Rename the suite so the test description matches what’s actually being tested.

describe('SpeakerName upload dialog (mobile)', () => {

• Files reviewed: 2/2 changed files
• Comments generated: 1
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.