diff --git a/crates/editor/src/editor.rs b/crates/editor/src/editor.rs index 3356d507cd..6cd9680d0e 100644 --- a/crates/editor/src/editor.rs +++ b/crates/editor/src/editor.rs @@ -1550,7 +1550,7 @@ impl Editor { } fn key_context(&self, cx: &AppContext) -> KeyContext { - let mut key_context = KeyContext::default(); + let mut key_context = KeyContext::new_with_defaults(); key_context.add("Editor"); let mode = match self.mode { EditorMode::SingleLine => "single_line", diff --git a/crates/extensions_ui/src/extensions_ui.rs b/crates/extensions_ui/src/extensions_ui.rs index 41d82a1405..25ef796784 100644 --- a/crates/extensions_ui/src/extensions_ui.rs +++ b/crates/extensions_ui/src/extensions_ui.rs @@ -700,7 +700,7 @@ impl ExtensionsPage { } fn render_search(&self, cx: &mut ViewContext) -> Div { - let mut key_context = KeyContext::default(); + let mut key_context = KeyContext::new_with_defaults(); key_context.add("BufferSearchBar"); let editor_border = if self.query_contains_error { diff --git a/crates/gpui/src/keymap/context.rs b/crates/gpui/src/keymap/context.rs index 6c22fa9fd6..6ac22d2162 100644 --- a/crates/gpui/src/keymap/context.rs +++ b/crates/gpui/src/keymap/context.rs @@ -25,6 +25,20 @@ impl<'a> TryFrom<&'a str> for KeyContext { } impl KeyContext { + /// Initialize a new [`KeyContext`] that contains an `os` key set to either `macos`, `linux`, `windows` or `unknown`. + pub fn new_with_defaults() -> Self { + let mut context = Self::default(); + #[cfg(target_os = "macos")] + context.set("os", "macos"); + #[cfg(target_os = "linux")] + context.set("os", "linux"); + #[cfg(target_os = "windows")] + context.set("os", "windows"); + #[cfg(not(any(target_os = "macos", target_os = "linux", target_os = "windows")))] + context.set("os", "unknown"); + context + } + /// Parse a key context from a string. /// The key context format is very simple: /// - either a single identifier, such as `StatusBar` diff --git a/crates/gpui/src/platform/keystroke.rs b/crates/gpui/src/platform/keystroke.rs index 5f3d75f72e..55f8658cd3 100644 --- a/crates/gpui/src/platform/keystroke.rs +++ b/crates/gpui/src/platform/keystroke.rs @@ -74,12 +74,7 @@ impl Keystroke { "alt" => alt = true, "shift" => shift = true, "fn" => function = true, - #[cfg(target_os = "macos")] - "cmd" => platform = true, - #[cfg(target_os = "linux")] - "super" => platform = true, - #[cfg(target_os = "windows")] - "win" => platform = true, + "cmd" | "super" | "win" => platform = true, _ => { if let Some(next) = components.peek() { if next.is_empty() && source.ends_with('-') { diff --git a/crates/project_panel/src/project_panel.rs b/crates/project_panel/src/project_panel.rs index b08f46b7e7..f592103b20 100644 --- a/crates/project_panel/src/project_panel.rs +++ b/crates/project_panel/src/project_panel.rs @@ -1698,7 +1698,7 @@ impl ProjectPanel { } fn dispatch_context(&self, cx: &ViewContext) -> KeyContext { - let mut dispatch_context = KeyContext::default(); + let mut dispatch_context = KeyContext::new_with_defaults(); dispatch_context.add("ProjectPanel"); dispatch_context.add("menu"); diff --git a/crates/search/src/buffer_search.rs b/crates/search/src/buffer_search.rs index 71ba5a65c3..0e0c33c265 100644 --- a/crates/search/src/buffer_search.rs +++ b/crates/search/src/buffer_search.rs @@ -188,7 +188,7 @@ impl Render for BufferSearchBar { let should_show_replace_input = self.replace_enabled && supported_options.replacement; let in_replace = self.replacement_editor.focus_handle(cx).is_focused(cx); - let mut key_context = KeyContext::default(); + let mut key_context = KeyContext::new_with_defaults(); key_context.add("BufferSearchBar"); if in_replace { key_context.add("in_replace"); diff --git a/crates/terminal_view/src/terminal_view.rs b/crates/terminal_view/src/terminal_view.rs index cc5c54eca9..6c3908305f 100644 --- a/crates/terminal_view/src/terminal_view.rs +++ b/crates/terminal_view/src/terminal_view.rs @@ -348,7 +348,7 @@ impl TerminalView { } fn dispatch_context(&self, cx: &AppContext) -> KeyContext { - let mut dispatch_context = KeyContext::default(); + let mut dispatch_context = KeyContext::new_with_defaults(); dispatch_context.add("Terminal"); let mode = self.terminal.read(cx).last_content.mode; diff --git a/crates/vim/src/state.rs b/crates/vim/src/state.rs index 82d4fc9af3..9ece818b16 100644 --- a/crates/vim/src/state.rs +++ b/crates/vim/src/state.rs @@ -194,7 +194,7 @@ impl EditorState { } pub fn keymap_context_layer(&self) -> KeyContext { - let mut context = KeyContext::default(); + let mut context = KeyContext::new_with_defaults(); context.set( "vim_mode", match self.mode { diff --git a/crates/workspace/src/dock.rs b/crates/workspace/src/dock.rs index 4dad415445..e6ece91f96 100644 --- a/crates/workspace/src/dock.rs +++ b/crates/workspace/src/dock.rs @@ -552,7 +552,7 @@ impl Dock { } fn dispatch_context() -> KeyContext { - let mut dispatch_context = KeyContext::default(); + let mut dispatch_context = KeyContext::new_with_defaults(); dispatch_context.add("Dock"); dispatch_context diff --git a/crates/workspace/src/pane.rs b/crates/workspace/src/pane.rs index c2d2b79961..de561320fa 100644 --- a/crates/workspace/src/pane.rs +++ b/crates/workspace/src/pane.rs @@ -1944,7 +1944,7 @@ impl FocusableView for Pane { impl Render for Pane { fn render(&mut self, cx: &mut ViewContext) -> impl IntoElement { - let mut key_context = KeyContext::default(); + let mut key_context = KeyContext::new_with_defaults(); key_context.add("Pane"); if self.active_item().is_none() { key_context.add("EmptyPane"); diff --git a/crates/workspace/src/workspace.rs b/crates/workspace/src/workspace.rs index 9c6e07b941..303323f735 100644 --- a/crates/workspace/src/workspace.rs +++ b/crates/workspace/src/workspace.rs @@ -3945,7 +3945,7 @@ struct DraggedDock(DockPosition); impl Render for Workspace { fn render(&mut self, cx: &mut ViewContext) -> impl IntoElement { - let mut context = KeyContext::default(); + let mut context = KeyContext::new_with_defaults(); context.add("Workspace"); let centered_layout = self.centered_layout && self.center.panes().len() == 1