Skip to content

refactor: remove unused SegmentControlsMobile and dead highlightAutoSegment prop#428

Merged
gtryus merged 1 commit into
developfrom
cleanup-unused-segment-controls-and-highlight-prop
Jul 14, 2026
Merged

refactor: remove unused SegmentControlsMobile and dead highlightAutoSegment prop#428
gtryus merged 1 commit into
developfrom
cleanup-unused-segment-controls-and-highlight-prop

Conversation

@nabalone

Copy link
Copy Markdown
Collaborator

Summary

Dead-code cleanup (no Jira ticket):

  • Delete SegmentControlsMobile — the component was never imported or rendered anywhere in the codebase.
  • Remove the highlightAutoSegment prop plumbing from WSAudioPlayer and WSAudioPlayerSegment. No caller ever passed highlightAutoSegment={true}, so the auto-segment button always rendered in its plain (non-highlighted) state. The prop was dead plumbing on both components. The auto-segment IconButton now drops its always-undefined variant.

Net: 74 deletions, no behavior change.

Test plan

  • npm run typecheck (from src/renderer) passes clean.
  • Auto-segment button in Mark Verses / Careful Speech still renders and functions identically (it was never highlighted before this change).

🤖 Generated with Claude Code

…egment prop

Delete the SegmentControlsMobile component, which was never imported or
rendered anywhere. Also remove the highlightAutoSegment prop plumbing from
WSAudioPlayer and WSAudioPlayerSegment: no caller ever set it to true, so the
auto-segment button always rendered in its plain (non-highlighted) state. The
button now drops the always-undefined variant.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nabalone nabalone marked this pull request as ready for review July 14, 2026 22:16
@gtryus gtryus merged commit fb0fe2a into develop Jul 14, 2026
2 checks passed
@gtryus gtryus deleted the cleanup-unused-segment-controls-and-highlight-prop branch July 14, 2026 22:40
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.

2 participants