Skip to content

Make trigger shortcut more obscure#41

Closed
BlueDrink9 wants to merge 1 commit intocursorless-dev:mainfrom
BlueDrink9:patch-1
Closed

Make trigger shortcut more obscure#41
BlueDrink9 wants to merge 1 commit intocursorless-dev:mainfrom
BlueDrink9:patch-1

Conversation

@BlueDrink9
Copy link
Copy Markdown

The current shortcut clashes with some extensions

This seems to work on my linux configuration with codium, whereas the f17 mapping didn't

The current shortcut [clashes with some extensions](cursorless-dev/cursorless#880 (comment))

This seems to work on my linux configuration with codium, whereas the f17 mapping didn't
@BlueDrink9
Copy link
Copy Markdown
Author

Matches with talonhub/community#1697

@pokey
Copy link
Copy Markdown
Member

pokey commented Jan 22, 2025

We need to tread carefully here, as this code has been through lots of paces to avoid various issues people have hit over time (eg keyboard layout / OS combos). We should a) test this on a few setups, and b) make sure we don't break backward compatibility, as users tend to update community very infrequently, whereas vscode extension pushes automatically

@BlueDrink9
Copy link
Copy Markdown
Author

BlueDrink9 commented Jan 22, 2025

That's about what I had hoped for. I can test on linux and windows, but if there are other iterations this has been through, I only saw the last couple.

To be honest, I'd be happy with just knowing how to override this in my own talon scripts

@BlueDrink9
Copy link
Copy Markdown
Author

Hadn't thought about keyboard layout, but I can see backticks being an issue there

@phillco
Copy link
Copy Markdown
Member

phillco commented Jan 23, 2025

We need to tread carefully here, as this code has been through lots of paces to avoid various issues people have hit over time (eg keyboard layout / OS combos).

I don't suppose we have a list of those factors, do we? I know the macOS v Windows shortcut is different for a reason but I can't remember why

@BlueDrink9 BlueDrink9 closed this Jul 12, 2025
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

Successfully merging this pull request may close these issues.

3 participants