Skip to content

Wrap window titles in switcher instead of truncating - #52

Open
pa wants to merge 1 commit into
gouwsxander:mainfrom
pa:feat/wrap-window-titles
Open

Wrap window titles in switcher instead of truncating#52
pa wants to merge 1 commit into
gouwsxander:mainfrom
pa:feat/wrap-window-titles

Conversation

@pa

@pa pa commented Jun 24, 2026

Copy link
Copy Markdown

Summary

  • Removes the fixed rowHeight: CGFloat = 44 constant and .frame(height: rowHeight) from CyclePanelRow
  • Replaces .lineLimit(1) with .fixedSize(horizontal: false, vertical: true) so long titles wrap onto a second line rather than being cut off

Motivation

Window titles for apps with multiple windows (especially browsers with multiple profiles open) often have the distinguishing information at the end — e.g. "Inbox (5) - Gmail - Google Chrome - Work". The previous fixed height + single-line truncation hid exactly that part. Wrapping keeps the full title visible without any parsing or app-specific logic.

Test plan

  • Open a browser with multiple windows across different profiles and trigger the switcher — confirm full title is visible
  • Confirm short titles still look normal (single line, correct padding)
  • Confirm scroll behaviour for >5 windows is unchanged

🤖 Generated with Claude Code

Removes the fixed row height and lineLimit(1) so long titles (e.g.
browser windows with a profile suffix) wrap onto a second line rather
than being cut off.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@pa
pa marked this pull request as ready for review June 24, 2026 14:57
@RetributionByRevenue

Copy link
Copy Markdown

this should be accepted into main, idk why it has not already

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