Merge pull request #104 from joshuamegnauth54/add-tab-keyboard-shortcuts

Port tab keyboard shortcuts over from cosmic-term and cosmic-files
pop-os · Jan 21, 2024 · d1f5fb8 · d1f5fb8
2 parents 4825864 + 196fe82
commit d1f5fb8
Show file tree

Hide file tree

Showing 2 changed files with 83 additions and 0 deletions.
diff --git a/src/key_bind.rs b/src/key_bind.rs
@@ -67,6 +67,17 @@ pub fn key_binds() -> HashMap<KeyBind, Action> {
     bind!([Ctrl, Shift], Z, Redo);
     bind!([Ctrl], S, Save);
     bind!([Ctrl], A, SelectAll);
+    bind!([Ctrl], Key1, TabActivate0);
+    bind!([Ctrl], Key2, TabActivate1);
+    bind!([Ctrl], Key3, TabActivate2);
+    bind!([Ctrl], Key4, TabActivate3);
+    bind!([Ctrl], Key5, TabActivate4);
+    bind!([Ctrl], Key6, TabActivate5);
+    bind!([Ctrl], Key7, TabActivate6);
+    bind!([Ctrl], Key8, TabActivate7);
+    bind!([Ctrl], Key9, TabActivate8);
+    bind!([Ctrl], Tab, TabNext);
+    bind!([Ctrl, Shift], Tab, TabPrev);
     bind!([Ctrl, Shift], G, ToggleGitManagement);
     bind!([Ctrl, Shift], F, ToggleProjectSearch);
     bind!([Ctrl], Comma, ToggleSettingsPage);

diff --git a/src/main.rs b/src/main.rs
@@ -171,6 +171,17 @@ pub enum Action {
     Redo,
     Save,
     SelectAll,
+    TabActivate0,
+    TabActivate1,
+    TabActivate2,
+    TabActivate3,
+    TabActivate4,
+    TabActivate5,
+    TabActivate6,
+    TabActivate7,
+    TabActivate8,
+    TabNext,
+    TabPrev,
     ToggleGitManagement,
     ToggleProjectSearch,
     ToggleSettingsPage,
@@ -196,6 +207,17 @@ impl Action {
             Self::Redo => Message::Redo,
             Self::Save => Message::Save,
             Self::SelectAll => Message::SelectAll,
+            Self::TabActivate0 => Message::TabActivateJump(0),
+            Self::TabActivate1 => Message::TabActivateJump(1),
+            Self::TabActivate2 => Message::TabActivateJump(2),
+            Self::TabActivate3 => Message::TabActivateJump(3),
+            Self::TabActivate4 => Message::TabActivateJump(4),
+            Self::TabActivate5 => Message::TabActivateJump(5),
+            Self::TabActivate6 => Message::TabActivateJump(6),
+            Self::TabActivate7 => Message::TabActivateJump(7),
+            Self::TabActivate8 => Message::TabActivateJump(8),
+            Self::TabNext => Message::TabNext,
+            Self::TabPrev => Message::TabPrev,
             Self::ToggleGitManagement => Message::ToggleContextPage(ContextPage::GitManagement),
             Self::ToggleProjectSearch => Message::ToggleContextPage(ContextPage::ProjectSearch),
             Self::ToggleSettingsPage => Message::ToggleContextPage(ContextPage::Settings),
@@ -272,10 +294,13 @@ pub enum Message {
     SystemThemeModeChange(cosmic_theme::ThemeMode),
     SyntaxTheme(usize, bool),
     TabActivate(segmented_button::Entity),
+    TabActivateJump(usize),
     TabChanged(segmented_button::Entity),
     TabClose(segmented_button::Entity),
     TabContextAction(segmented_button::Entity, Action),
     TabContextMenu(segmented_button::Entity, Option<Point>),
+    TabNext,
+    TabPrev,
     TabSetCursor(segmented_button::Entity, Cursor),
     TabWidth(u16),
     Todo,
@@ -1661,6 +1686,21 @@ impl Application for App {
                 self.tab_model.activate(entity);
                 return self.update_tab();
             }
+            Message::TabActivateJump(pos) => {
+                // Length is always at least one, so there shouldn't be a division by zero
+                let len = self.tab_model.iter().count();
+                // Indices 1 to 8 jumps to tabs 1-8 while 9 jumps to the last
+                let pos = if pos >= 8 || pos > len - 1 {
+                    len - 1
+                } else {
+                    pos % len
+                };
+
+                let entity = self.tab_model.iter().nth(pos);
+                if let Some(entity) = entity {
+                    return self.update(Message::TabActivate(entity));
+                }
+            }
             Message::TabChanged(entity) => {
                 if let Some(Tab::Editor(tab)) = self.tab_model.data::<Tab>(entity) {
                     let mut title = tab.title();
@@ -1703,6 +1743,38 @@ impl Application for App {
                     tab.context_menu = position_opt;
                 }
             }
+            Message::TabNext => {
+                let len = self.tab_model.iter().count();
+                // Next tab position. Wraps around to 0 (the first tab) if the last tab is active.
+                let pos = self
+                    .tab_model
+                    .position(self.tab_model.active())
+                    .map(|i| (i as usize + 1) % len)
+                    .expect("at least one tab is always open");
+
+                let entity = self.tab_model.iter().nth(pos);
+                if let Some(entity) = entity {
+                    return self.update(Message::TabActivate(entity));
+                }
+            }
+            Message::TabPrev => {
+                let pos = self
+                    .tab_model
+                    .position(self.tab_model.active())
+                    .and_then(|i| (i as usize).checked_sub(1))
+                    .unwrap_or_else(|| {
+                        self.tab_model
+                            .iter()
+                            .count()
+                            .checked_sub(1)
+                            .unwrap_or_default()
+                    });
+
+                let entity = self.tab_model.iter().nth(pos);
+                if let Some(entity) = entity {
+                    return self.update(Message::TabActivate(entity));
+                }
+            }
             Message::TabSetCursor(entity, cursor) => {
                 if let Some(Tab::Editor(tab)) = self.tab_model.data::<Tab>(entity) {
                     let mut editor = tab.editor.lock().unwrap();