Skip to content

[feat] Make hotkey for exiting subgraphs configurable in user keybindings #4818

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Aug 11, 2025

Conversation

christian-byrne
Copy link
Contributor

@christian-byrne christian-byrne commented Aug 7, 2025

Fixes #4208

Makes the Escape hotkey for exiting subgraphs configurable through the keybinding system. Users can now rebind this key to their preference while maintaining proper dialog handling priority.

┆Issue is synchronized with this Notion page by Unito

@christian-byrne christian-byrne requested review from a team as code owners August 7, 2025 20:02
@christian-byrne christian-byrne force-pushed the feat/configurable-escape-subgraph branch from 18fad19 to 9ca8a64 Compare August 7, 2025 20:16
Copy link

github-actions bot commented Aug 7, 2025

⚠️ Warnings

⚠️ Warning: Visual Documentation Missing

If this PR changes user-facing behavior, visual proof (screen recording or screenshot) is required. PRs without applicable visual documentation may not be reviewed until provided.
You can add it by:

  • GitHub: Drag & drop media directly into the PR description

  • YouTube: Include a link to a short demo

Fixes #4208 - Escape hotkey is now configurable through the keybinding system.
Added Exit Subgraph command that can be bound to any key combination.
Escape key prioritizes closing dialogs before exiting subgraphs.
@christian-byrne christian-byrne force-pushed the feat/configurable-escape-subgraph branch from f455df3 to a8800b8 Compare August 9, 2025 19:21
The test was overly mocked and provided little value beyond what's already
covered by keybindingService tests and browser tests.
@christian-byrne christian-byrne force-pushed the feat/configurable-escape-subgraph branch from 98d03c8 to 4996a15 Compare August 9, 2025 22:12
@christian-byrne christian-byrne added needs-backport Fix/change that needs to be cherry-picked to the current feature freeze branch area:subgraph 1.25 area:hotkeys labels Aug 11, 2025
Instead of navigating through the UI, directly set keybinding settings
which is more reliable and faster.
@christian-byrne christian-byrne changed the title [feat] Make Escape key configurable for exiting subgraphs [feat] Make hotkey for exiting subgraphs configurable in user keybindings Aug 11, 2025
@christian-byrne christian-byrne merged commit 71a4319 into main Aug 11, 2025
13 checks passed
@christian-byrne christian-byrne deleted the feat/configurable-escape-subgraph branch August 11, 2025 17:22
github-actions bot pushed a commit that referenced this pull request Aug 11, 2025
@comfy-pr-bot
Copy link
Member

@christian-byrne Successfully backported to #4914

This was referenced Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.25 area:hotkeys area:subgraph needs-backport Fix/change that needs to be cherry-picked to the current feature freeze branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Option to hide subgraph tree + hotkey to go the previous subgraph level
2 participants