-
-
Notifications
You must be signed in to change notification settings - Fork 808
keyboard shortcuts are not saved #1748
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
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Captchas
What happened?
Shortcuts are not saved after changing them in keyboard shortcuts settings window.
I've been looking at this file: ZenKeyboardshortcuts.mjs and notices that there are 2 variables for the default saved shortcuts:
_savedShortcuts
__savedShortcuts
When the default shortcuts are loaded they are saved into the one with double underscore, while the rest of the functions read/write to the one with single _
I changed the mode of file: zen-keyboard-shortcuts.json under default profile to read only after editing the file with the required shortcuts and they got loaded properly in the settings page, however the new shortcut didn't take effect as seems browser memory is reading from the runtime default settings not the one saved from the previous session.
I haven't dug deeper into the code but thought that might help mitigating the issue.
https://github.com/zen-browser/components/blob/36e62781e27c19fade2df4d6a63e9062862ce7b0/src/ZenKeyboardShortcuts.mjs#L201
Reproducible?
Version
1.0.1-a.5 (64-bit)
What platform are you seeing the problem on?
macOS - aarch64
Relevant log output
No response
The text was updated successfully, but these errors were encountered: