Skip to content

Conversation

dkr91
Copy link

@dkr91 dkr91 commented Aug 9, 2025

Adds Customizable Shortcuts as seen in #5079

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. Feature labels Aug 9, 2025
'zen_glance-shortcut-split'
)
);
newShortcutList.push(
Copy link
Contributor

Choose a reason for hiding this comment

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

Esc is already a shortcut, purposefully done on the content process. I'd do the same with the other shortcuts

Copy link
Author

Choose a reason for hiding this comment

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

What content process. were would I need to add 'ESC'ape instead? thanks for the input.

Copy link
Contributor

Choose a reason for hiding this comment

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

The ESC key is arleady added, im saying that the other shortcuts should be added there as well.

Choose a reason for hiding this comment

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

Are you referring to this line? The Esc key is hardcoded in this case.

How would you add the other hotkeys while making the hotkeys customiseable in ZenGlanceManager.mjs?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants