Skip to content
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

regression in command palette: text commands not shown #5707

Closed
tessus opened this issue Nov 10, 2021 · 4 comments
Closed

regression in command palette: text commands not shown #5707

tessus opened this issue Nov 10, 2021 · 4 comments
Labels
bug It's a bug desktop All desktop platforms high High priority issues

Comments

@tessus
Copy link
Collaborator

tessus commented Nov 10, 2021

Environment

Joplin 2.6.1 (dev, darwin)

Client ID: e354a932e6e143c782264aad4112b674
Sync Version: 3
Profile Version: 40
Keychain Supported: Yes

Revision: 725c79d (dev)

Steps to reproduce

  1. open Command Palette
  2. try to find any commands in the Edit menu e.g. bold, code, .... (internally textBold, or actually all textXXXXX commands)
  3. not found

Describe what you expected to happen

The text commands should be shown in the command palette. I believe this is a regression. I am fairly certain this worked before.

Logfile

No errors in console or log.

@tessus tessus added the bug It's a bug label Nov 10, 2021
@laurent22
Copy link
Owner

Probably caused by this commit: 23996e0

I guess text commands are not quite integrated to the command system, and in particular their "enabled" property is not set correctly. Not sure what's the fix - either fix the text commands or revert the above commit I guess.

@tessus
Copy link
Collaborator Author

tessus commented Nov 14, 2021

I'm not sure how to fix the text commands. At one point the code became so abstracted, thus one has to look in so many different files to verify a single thing. It's very confusing when you don't look at the code every single day.

@laurent22
Copy link
Owner

Probably related: #5752

@laurent22 laurent22 added desktop All desktop platforms high High priority issues labels Nov 19, 2021
@SndChaser
Copy link

I think I have a related issue (if not I can open a new issue): assigning a shortcut to Insert Date Time in Tools->Options->Keyboard Shortcuts doesn't work. IE, you can make the key assignment, but using it in the editor produces no output. I assume that this has to do with changes made for the Command Palette (which I also note doesn't have an InsertDateTime command).

Environment: Linux Manjaro (Kernel 5.13.19) / XFCE 4.16
Joplin 2.5.12 (AUR AppImage)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug It's a bug desktop All desktop platforms high High priority issues
Projects
None yet
Development

No branches or pull requests

3 participants