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

Jetbrains Keybind: Stop binding shift-shift by default #14252

Closed
wants to merge 1 commit into from

Conversation

notpeter
Copy link
Member

Release Notes:

  • Removed "shift shift" from Jetbrains default keybinds.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jul 11, 2024
@notpeter notpeter mentioned this pull request Jul 11, 2024
1 task
@notpeter notpeter force-pushed the disable_shift_shift branch from b7f1c22 to 0de3293 Compare July 11, 2024 21:05
@ConradIrwin
Copy link
Member

Seems like having it is better than not having it, but maybe we need to document the way zed does command palette as a known issue for jetbrains

@notpeter
Copy link
Member Author

notpeter commented Jul 11, 2024

@ConradIrwin I disagree. When I used PyCharm, I used it exclusively for file_finder::Toggle functionality, but most of my team used it for project_symbols::Toggle and I didn't know anyone who used it for command_palette::Toggle functionality.

I think choosing any one as a default one means that the majority of JetBrains users will be sad the majority of the time they try and use it. And in that case, doing nothing might better than doing something wrong.

We don't have a Search Everything function like JetBrains and that's ok.

@aohanhongzhi
Copy link
Contributor

keep it . and waiting for "Search Everything" function

@notpeter notpeter closed this Jul 24, 2024
@notpeter notpeter deleted the disable_shift_shift branch July 24, 2024 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants