Skip to content

Trim long paths in Reopen menu and tighten menu spacing#11050

Merged
niksedk merged 3 commits into
mainfrom
reopen-menu-truncate-and-tighten
May 19, 2026
Merged

Trim long paths in Reopen menu and tighten menu spacing#11050
niksedk merged 3 commits into
mainfrom
reopen-menu-truncate-and-tighten

Conversation

@niksedk
Copy link
Copy Markdown
Member

@niksedk niksedk commented May 19, 2026

Summary

  • Reopen submenu entries with long paths were being clipped. Switched the TextBlock to TextTrimming.PrefixCharacterEllipsis with MaxWidth = 600 so the directory prefix gets truncated as …\folder\file.srt while the filename stays visible (tooltip still shows the full path).
  • Dropped the menu-wide MenuItem padding from (10, 3) to (10, 1) for a denser, tighter menu list.

Test plan

  • Open the File → Reopen menu with a recent file whose path is longer than the menu — verify the start is truncated with and the filename is fully visible.
  • Hover a truncated entry — tooltip shows the full path.
  • Visually confirm all top-level menus (File, Edit, Tools, …) now have slightly less vertical spacing between items.

🤖 Generated with Claude Code

niksedk and others added 3 commits May 19, 2026 15:34
Long file paths in the Reopen submenu were being clipped, so trim the
directory prefix with "…" (filename stays visible, tooltip shows the
full path). Also drops top-level menu padding one notch so the menu
reads as a denser list.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The Fluent theme enforces a default MinHeight on MenuItem, which made
the padding change a no-op visually. Lowering MinHeight to 22 lets the
tighter padding take effect.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@niksedk niksedk merged commit c86216e into main May 19, 2026
1 of 3 checks passed
@niksedk niksedk deleted the reopen-menu-truncate-and-tighten branch May 19, 2026 15:03
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.

1 participant