Skip to content

platform: disable more UI controls#459

Merged
irahopkinson merged 1 commit intomainfrom
history-shortcuts
Mar 10, 2026
Merged

platform: disable more UI controls#459
irahopkinson merged 1 commit intomainfrom
history-shortcuts

Conversation

@irahopkinson
Copy link
Copy Markdown
Collaborator

@irahopkinson irahopkinson commented Mar 10, 2026

  • if hasExternalUI then undo & redo keyboard shortcuts are disabled.
  • if hasExternalUI then the marker menu is disabled.

Open with Devin

This change is Reviewable

- if `hasExternalUI` then undo & redo keyboard shortcuts are disabled.
- if `hasExternalUI` then the marker menu is disabled.
@codesandbox
Copy link
Copy Markdown

codesandbox bot commented Mar 10, 2026

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@irahopkinson irahopkinson enabled auto-merge (squash) March 10, 2026 07:35
Copy link
Copy Markdown
Contributor

@tjcouch-sil tjcouch-sil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Copy Markdown
Contributor

@tjcouch-sil tjcouch-sil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@tjcouch-sil reviewed 5 files and all commit messages, made 1 comment, and resolved 1 discussion.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved.

@irahopkinson irahopkinson merged commit 4b5802f into main Mar 10, 2026
7 checks passed
@irahopkinson irahopkinson deleted the history-shortcuts branch March 10, 2026 13:56
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