Skip to content

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

Closed
3 of 4 tasks
samhuss opened this issue Sep 26, 2024 · 5 comments
Closed
3 of 4 tasks

keyboard shortcuts are not saved #1748

samhuss opened this issue Sep 26, 2024 · 5 comments

Comments

@samhuss
Copy link

samhuss commented Sep 26, 2024

Captchas

  • I have read the instructions.
  • I have searched existing issues and avoided creating duplicates.
  • I am not filing an enhancement request.

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?

  • I have checked that this issue cannot be reproduced on Mozilla Firefox.

Version

1.0.1-a.5 (64-bit)

What platform are you seeing the problem on?

macOS - aarch64

Relevant log output

No response

Copy link

linear bot commented Sep 26, 2024

@R3flector
Copy link

same on arch

@wetsocksnsleeves
Copy link

Can confirm the same on Windows 10

@thebiblelover7
Copy link

Seems like a duplicate of #1715 to me.

@mauro-balades
Copy link
Member

mauro-balades commented Sep 28, 2024

This has been fixed for the next release! Also, it's a duplicate of #1715, you can follow the issue there

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

No branches or pull requests

5 participants