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

combobulate-python conflicts with which-key #77

Open
jeff-phil opened this issue Jan 11, 2024 · 1 comment
Open

combobulate-python conflicts with which-key #77

jeff-phil opened this issue Jan 11, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@jeff-phil
Copy link

When using combobulate-python the pop-up for a which-key prefix does not always show up.

Steps to reproduce:

  1. Open a python file with combobulate and which-key enabled.
  2. Type a few letters such as prin, pause for 1 second, and then type C-c for the mode-specific-map.
  3. Notice which-key popup does not appear.
  4. Now go to a new blank line, repeat step 2 above, and notice which-key popup does appear.

The problem is this-command on Line 147 of combobulate-python.el is being set, and then Line 2685 of which-key.el sees the this-command and assumes there is an active command execution and then won't show the popup.

I thought (read: assumed) this-command was meant to be a transactional indicator for other packages to observe or react to a current running command in a package.

Since it looks like it may be only used internally by combobulate-python, do you think using local variables (combob-this-command & combob-last-command) would be better?


On a side-note, turning off combobulate-mode for does not reset back indent-region-function variable.

@mickeynp mickeynp added the bug Something isn't working label Jan 12, 2024
@mickeynp
Copy link
Owner

Altering this-command is perfectly fine. It's there so combobulate can tell when you tab multiple times in a row. However, I do intend to move it to using the proffer system that other parts of Combobulate uses, so it will eventually be replaced.

On a side-note, turning off combobulate-mode for does not reset back indent-region-function variable.

Thanks for telling me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants