Skip to content

Add double-click to toggle checkbox in FixCommonErrors#10

Open
ivandrofly wants to merge 35 commits intomainfrom
feature/double-click-toggle-checkbox
Open

Add double-click to toggle checkbox in FixCommonErrors#10
ivandrofly wants to merge 35 commits intomainfrom
feature/double-click-toggle-checkbox

Conversation

@ivandrofly
Copy link
Copy Markdown
Owner

Summary

  • Double-clicking a row in the rules grid (step 1) now toggles the enabled checkbox for that rule
  • Double-clicking a row in the fixes grid (step 2) now toggles the apply checkbox for that fix

Test plan

  • Open Fix Common Errors dialog
  • In step 1, double-click a rule row and verify the checkbox toggles
  • In step 2, double-click a fix row and verify the checkbox toggles
  • Verify single-click on checkbox still works as before

🤖 Generated with Claude Code

ivandrofly and others added 30 commits March 13, 2026 22:20
…d unit tests

Added `Save` and `LoadSubtitle` methods for binary persistence. Created comprehensive tests to ensure proper functionality for saving and loading subtitles, timecodes, and multi-line paragraphs while verifying data integrity.
…-file

Update `LibSE.csproj`: fix incorrect `TargetFrameworks` property usage
…-caption-old-save-support

Add save support to CheetahCaptionOld format
ivandrofly and others added 5 commits March 19, 2026 12:36
Extended functionality with a new "Insert Line" command targeting empty grids. Adjusted the subtitle grid context menu binding to replace redundant checks with state-specific visibility logic. Enhanced empty area interactivity by setting the context menu on the drop host.

Closes SubtitleEdit#10351
Refactored `override public` to `public override` across multiple files for consistency and adherence to coding standards.
…line

Add "Insert Line" feature and improve subtitle grid context menu logic
…tring-override-order

Update ToString override declarations to align with C# conventions
Double-clicking a row in the rules grid (step 1) or fixes grid (step 2)
now toggles the checkbox for that row.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ivandrofly ivandrofly force-pushed the feature/double-click-toggle-checkbox branch from c378ab7 to d433ecd Compare March 20, 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.

2 participants