Skip to content

Add a Source View button to the main toolbar#11608

Merged
niksedk merged 2 commits into
SubtitleEdit:mainfrom
muaz978:feature/sourceview-toolbar-button
Jun 15, 2026
Merged

Add a Source View button to the main toolbar#11608
niksedk merged 2 commits into
SubtitleEdit:mainfrom
muaz978:feature/sourceview-toolbar-button

Conversation

@muaz978

@muaz978 muaz978 commented Jun 14, 2026

Copy link
Copy Markdown

Fixes #11605

Source View only existed as a command in SE5 (no toolbar entry), so the toggle button from SE4 was missing.

  • Adds a toolbar button using the existing SourceView theme icon that runs ShowSourceViewCommand.
  • Gated by a new ToolbarShowSourceView appearance flag (default on) with a matching "Show source view icon" checkbox in Settings > Appearance, like the other toolbar icons.

Builds clean (0/0). Test build provided so the icon/placement can be checked.

Source View existed only as a command (no toolbar entry), so the toggle
button users had in SE4 was missing. Add a toolbar button - using the
existing SourceView theme icon - that runs ShowSourceViewCommand, gated by a
new ToolbarShowSourceView appearance flag (default on) with a matching
"Show source view icon" checkbox in Settings > Appearance.
@niksedk

niksedk commented Jun 14, 2026

Copy link
Copy Markdown
Member

No new language tags for now, so this must wait

@niksedk

niksedk commented Jun 14, 2026

Copy link
Copy Markdown
Member

Also, is the default value false?

- Default ToolbarShowSourceView to false so the toolbar is unchanged for
  existing users unless they opt in.
- Drop the new language tags (SourceViewHint, ShowToolbarSourceView) and reuse
  the existing Options.Shortcuts.SourceView ("Source view") string for both the
  toolbar tooltip/automation name and the settings checkbox, so no new
  translation tags are introduced.
@muaz978

muaz978 commented Jun 14, 2026

Copy link
Copy Markdown
Author

Thanks - addressed both:

  • Default is now false (toolbar unchanged unless the user opts in).
  • Removed the new language tags; the button tooltip, accessibility name and the settings checkbox now reuse the existing Options.Shortcuts.SourceView ("Source view") string, so no new translation tags are added. Pushed to this branch.

@niksedk niksedk merged commit cbc4734 into SubtitleEdit:main Jun 15, 2026
2 checks passed
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.

Source View has no toolbar button (toggle gone from SE4)

2 participants