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

Keybinding CMD+K not working #16763

Open
1 task done
garrettg123 opened this issue Aug 23, 2024 · 2 comments
Open
1 task done

Keybinding CMD+K not working #16763

garrettg123 opened this issue Aug 23, 2024 · 2 comments
Labels
diagnostics Feedback for diagnostics, error messages, logs, etc keymap / key binding Feedback for keyboard shortcuts, key mapping, etc support User support (non-defect troubleshooting, documentation, etc)

Comments

@garrettg123
Copy link

Check for existing issues

  • Completed

Describe the bug / provide steps to reproduce it

keymap.json

// Zed keymap
//
// For information on binding keys, see the Zed
// documentation: https://zed.dev/docs/key-bindings
//
// To see the default key bindings run `zed: open default keymap`
// from the command palette.
[
  {
    "context": "Workspace",
    "bindings": {
      // "shift shift": "file_finder::Toggle"
    }
  },
  {
    "context": "Editor",
    "bindings": {
      // "j k": ["workspace::SendKeystrokes", "escape"]
      "cmd-k n": ["editor::GoToDiagnostic"]
    }
  }
]

Environment

Zed: v0.149.5 (Zed)
OS: macOS 14.5.0
Memory: 32 GiB
Architecture: aarch64

If applicable, add mockups / screenshots to help explain present your vision of the feature

No response

If applicable, attach your Zed.log file to this issue.

Zed.log
@garrettg123 garrettg123 added admin read Pending admin review bug [core label] triage Maintainer needs to classify the issue labels Aug 23, 2024
@notpeter
Copy link
Member

notpeter commented Aug 23, 2024

That keybinding works for me.
Did you perhaps want diagnostics::Deploy (toggles focus to the Diagnostics panel)?

editor::GoToDiagnostic (goes to the next diagnostic error/warning in the current file, pairs with editor::GoToPrevDiagnostic to go to the previous)?

@notpeter notpeter added keymap / key binding Feedback for keyboard shortcuts, key mapping, etc diagnostics Feedback for diagnostics, error messages, logs, etc support User support (non-defect troubleshooting, documentation, etc) and removed bug [core label] triage Maintainer needs to classify the issue admin read Pending admin review labels Aug 23, 2024
@garrettg123
Copy link
Author

No, I mean the key binding doesn't do anything, regardless of what I make the action. Maybe I'm putting in the wrong syntax for cmd or maybe these are silently failing?

// Zed keymap
//
// For information on binding keys, see the Zed
// documentation: https://zed.dev/docs/key-bindings
//
// To see the default key bindings run `zed: open default keymap`
// from the command palette.
[
  {
    "context": "Workspace",
    "bindings": {
      // "shift shift": "file_finder::Toggle"
    }
  },
  {
    "context": "Editor",
    "bindings": {
      // "j k": ["workspace::SendKeystrokes", "escape"]
      "cmd-k n": ["editor::GoToDiagnostic"],
      "ctrl-cmd-w": ["pane::CloseInactiveItems"],
      "cmd-@": ["editor::FindAllReferences"]
    }
  }
]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
diagnostics Feedback for diagnostics, error messages, logs, etc keymap / key binding Feedback for keyboard shortcuts, key mapping, etc support User support (non-defect troubleshooting, documentation, etc)
Projects
None yet
Development

No branches or pull requests

2 participants