Skip to content

Subtitle grid: more "Selected lines" context menu items - #14798

Merged
niksedk merged 1 commit into
mainfrom
claude/subtitlegrid-context-menu-1f3fcd
Sep 12, 2026
Merged

Subtitle grid: more "Selected lines" context menu items#14798
niksedk merged 1 commit into
mainfrom
claude/subtitlegrid-context-menu-1f3fcd

Conversation

@niksedk

@niksedk niksedk commented Sep 12, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds Adjust durations..., Apply duration limits... and Change formatting... to the subtitle grid's Selected lines context submenu, each applied to the selected rows only.
  • Adjust durations and Apply duration limits keep walking the whole grid, so a selected line is capped against its real next line rather than the next selected line.
  • Fixes the submenu's Text to speech item, which ran the whole-file command. It now opens the TTS window with just the selected lines (plus their original lines when an original is shown), and a TTS merge can only swallow rows inside the selection.

Test plan

  • AdjustDurationViewModelTests and ApplyDurationLimitsViewModelTests pass, including two new tests for the selected-rows filters.
  • UI project builds.
  • Note: ShorteningStillWorksWhenTheUnusedMinimumIsHigherThanTheMaximum fails on main as well (pre-existing, unrelated).

🤖 Generated with Claude Code

Add "Adjust durations...", "Apply duration limits..." and "Change
formatting..." to the grid's Selected lines submenu, each working on the
selected rows only. Adjust durations and Apply duration limits still walk
the whole grid so a selected line is capped against its real neighbour,
not the next line that happened to be selected.

The submenu's "Text to speech" item ran the whole-file command; it now
opens the TTS window with just the selected lines (and their original
lines when an original is shown). A TTS merge can then only swallow rows
inside the selection, since lines adjacent in the selection need not be
adjacent in the grid.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@niksedk
niksedk merged commit 6476d4d into main Sep 12, 2026
1 check passed
@niksedk
niksedk deleted the claude/subtitlegrid-context-menu-1f3fcd branch September 12, 2026 07:49
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