Merge pull request #2160 from zed-industries/remove-toggle-right-sidebar-command

Joseph T. Lyons created 2 years ago

Remove toggle right sidebar command

Change summary

assets/keymaps/default.json       | 7 -------
crates/workspace/src/workspace.rs | 4 ----
crates/zed/src/menus.rs           | 4 ----
3 files changed, 15 deletions(-)

Detailed changes

assets/keymaps/default.json

@@ -430,13 +430,6 @@
             "cmd-enter": "project_search::SearchInNew"
         }
     },
-    {
-        "context": "Workspace",
-        "bindings": {
-            "shift-escape": "dock::FocusDock",
-            "cmd-shift-b": "workspace::ToggleRightSidebar"
-        }
-    },
     {
         "bindings": {
             "cmd-shift-k cmd-shift-right": "dock::AnchorDockRight",

crates/workspace/src/workspace.rs

@@ -96,7 +96,6 @@ actions!(
         ActivateNextPane,
         FollowNextCollaborator,
         ToggleLeftSidebar,
-        ToggleRightSidebar,
         NewTerminal,
         NewSearch,
         Feedback,
@@ -230,9 +229,6 @@ pub fn init(app_state: Arc<AppState>, cx: &mut MutableAppContext) {
     cx.add_action(|workspace: &mut Workspace, _: &ToggleLeftSidebar, cx| {
         workspace.toggle_sidebar(SidebarSide::Left, cx);
     });
-    cx.add_action(|workspace: &mut Workspace, _: &ToggleRightSidebar, cx| {
-        workspace.toggle_sidebar(SidebarSide::Right, cx);
-    });
     cx.add_action(Workspace::activate_pane_at_index);
 
     cx.add_action(Workspace::split_pane_with_item);

crates/zed/src/menus.rs

@@ -219,10 +219,6 @@ pub fn menus() -> Vec<Menu<'static>> {
                     name: "Toggle Left Sidebar",
                     action: Box::new(workspace::ToggleLeftSidebar),
                 },
-                MenuItem::Action {
-                    name: "Toggle Right Sidebar",
-                    action: Box::new(workspace::ToggleRightSidebar),
-                },
                 MenuItem::Submenu(Menu {
                     name: "Editor Layout",
                     items: vec![