diff --git a/i18n/en/cosmic_term.ftl b/i18n/en/cosmic_term.ftl index 2da7794..229ca82 100644 --- a/i18n/en/cosmic_term.ftl +++ b/i18n/en/cosmic_term.ftl @@ -44,6 +44,11 @@ find = Find ## View view = View +zoom-in = Larger text +zoom-reset = Default text size +zoom-out = Smaller text +next-tab = Next tab +previous-tab = Previous tab menu-settings = Settings... # Context menu diff --git a/src/key_bind.rs b/src/key_bind.rs index e6b2a58..6d3a7b4 100644 --- a/src/key_bind.rs +++ b/src/key_bind.rs @@ -56,6 +56,8 @@ pub fn key_binds() -> HashMap { bind!([Ctrl, Shift], A, SelectAll); bind!([Ctrl, Shift], C, Copy); bind!([Ctrl, Shift], F, Find); + bind!([Ctrl, Shift], N, WindowNew); + bind!([Ctrl, Shift], Q, WindowClose); bind!([Ctrl, Shift], T, TabNew); bind!([Ctrl, Shift], V, Paste); bind!([Ctrl, Shift], W, TabClose); diff --git a/src/main.rs b/src/main.rs index 7317058..ffa4198 100644 --- a/src/main.rs +++ b/src/main.rs @@ -162,7 +162,7 @@ pub struct Flags { term_config: TermConfig, } -#[derive(Clone, Copy, Debug)] +#[derive(Clone, Copy, Debug, Eq, PartialEq)] pub enum Action { Copy, Find, @@ -190,15 +190,17 @@ pub enum Action { TabNew, TabNext, TabPrev, + WindowClose, + WindowNew, ZoomIn, ZoomOut, ZoomReset, } impl Action { - pub fn message(self, entity: segmented_button::Entity) -> Message { + pub fn message(self, entity_opt: Option) -> Message { match self { - Action::Copy => Message::Copy(Some(entity)), + Action::Copy => Message::Copy(entity_opt), Action::Find => Message::Find(true), Action::PaneFocusDown => Message::PaneFocusAdjacent(pane_grid::Direction::Down), Action::PaneFocusLeft => Message::PaneFocusAdjacent(pane_grid::Direction::Left), @@ -207,8 +209,8 @@ impl Action { Action::PaneSplitHorizontal => Message::PaneSplit(pane_grid::Axis::Horizontal), Action::PaneSplitVertical => Message::PaneSplit(pane_grid::Axis::Vertical), Action::PaneToggleMaximized => Message::PaneToggleMaximized, - Action::Paste => Message::Paste(Some(entity)), - Action::SelectAll => Message::SelectAll(Some(entity)), + Action::Paste => Message::Paste(entity_opt), + Action::SelectAll => Message::SelectAll(entity_opt), Action::Settings => Message::ToggleContextPage(ContextPage::Settings), Action::ShowHeaderBar(show_headerbar) => Message::ShowHeaderBar(show_headerbar), Action::TabActivate0 => Message::TabActivateJump(0), @@ -220,10 +222,12 @@ impl Action { Action::TabActivate6 => Message::TabActivateJump(6), Action::TabActivate7 => Message::TabActivateJump(7), Action::TabActivate8 => Message::TabActivateJump(8), - Action::TabClose => Message::TabClose(None), + Action::TabClose => Message::TabClose(entity_opt), Action::TabNew => Message::TabNew, Action::TabNext => Message::TabNext, Action::TabPrev => Message::TabPrev, + Action::WindowClose => Message::WindowClose, + Action::WindowNew => Message::WindowNew, Action::ZoomIn => Message::ZoomIn, Action::ZoomOut => Message::ZoomOut, Action::ZoomReset => Message::ZoomReset, @@ -997,12 +1001,9 @@ impl Application for App { } }, Message::Key(modifiers, key_code) => { - if let Some(tab_model) = self.pane_model.active() { - let entity = tab_model.active(); - for (key_bind, action) in self.key_binds.iter() { - if key_bind.matches(modifiers, key_code) { - return self.update(action.message(entity)); - } + for (key_bind, action) in self.key_binds.iter() { + if key_bind.matches(modifiers, key_code) { + return self.update(action.message(None)); } } } @@ -1205,7 +1206,7 @@ impl Application for App { terminal.context_menu = None; } // Run action's message - return self.update(action.message(entity)); + return self.update(action.message(Some(entity))); } _ => {} } @@ -1371,7 +1372,7 @@ impl Application for App { } fn header_start(&self) -> Vec> { - vec![menu_bar().into()] + vec![menu_bar(&self.key_binds).into()] } /// Creates a view after each update. diff --git a/src/menu.rs b/src/menu.rs index 5c693b0..44dd2c4 100644 --- a/src/menu.rs +++ b/src/menu.rs @@ -14,8 +14,9 @@ use cosmic::{ }, Element, }; +use std::collections::HashMap; -use crate::{fl, Action, Config, ContextPage, Message}; +use crate::{fl, Action, Config, KeyBind, Message}; macro_rules! menu_button { ($($x:expr),+ $(,)?) => ( @@ -59,7 +60,7 @@ pub fn context_menu<'a>(config: &Config, entity: segmented_button::Entity) -> El menu_action(fl!("pane-toggle-maximize"), Action::PaneToggleMaximized), horizontal_rule(1), menu_action(fl!("new-tab"), Action::TabNew), - menu_action(fl!("settings"), Action::Settings), + menu_action(fl!("menu-settings"), Action::Settings), menu_checkbox( fl!("show-headerbar"), config.show_headerbar, @@ -84,7 +85,7 @@ pub fn context_menu<'a>(config: &Config, entity: segmented_button::Entity) -> El .into() } -pub fn menu_bar<'a>() -> Element<'a, Message> { +pub fn menu_bar<'a>(key_binds: &HashMap) -> Element<'a, Message> { //TODO: port to libcosmic let menu_root = |label| { widget::button(widget::text(label)) @@ -92,20 +93,24 @@ pub fn menu_bar<'a>() -> Element<'a, Message> { .style(theme::Button::MenuRoot) }; - let find_key = |message: &Message| -> String { - //TODO: hotkey config + let find_key = |action: &Action| -> String { + for (key_bind, key_action) in key_binds.iter() { + if action == key_action { + return key_bind.to_string(); + } + } String::new() }; - let menu_item = |label, message| { - let key = find_key(&message); + let menu_item = |label, action| { + let key = find_key(&action); MenuTree::new( menu_button!( widget::text(label), horizontal_space(Length::Fill), widget::text(key) ) - .on_press(message), + .on_press(action.message(None)), ) }; @@ -113,30 +118,40 @@ pub fn menu_bar<'a>() -> Element<'a, Message> { MenuTree::with_children( menu_root(fl!("file")), vec![ - menu_item(fl!("new-tab"), Message::TabNew), - menu_item(fl!("new-window"), Message::WindowNew), + menu_item(fl!("new-tab"), Action::TabNew), + menu_item(fl!("new-window"), Action::WindowNew), MenuTree::new(horizontal_rule(1)), - menu_item(fl!("close-tab"), Message::TabClose(None)), + menu_item(fl!("close-tab"), Action::TabClose), MenuTree::new(horizontal_rule(1)), - menu_item(fl!("quit"), Message::WindowClose), + menu_item(fl!("quit"), Action::WindowClose), ], ), MenuTree::with_children( menu_root(fl!("edit")), vec![ - menu_item(fl!("copy"), Message::Copy(None)), - menu_item(fl!("paste"), Message::Paste(None)), - menu_item(fl!("select-all"), Message::SelectAll(None)), + menu_item(fl!("copy"), Action::Copy), + menu_item(fl!("paste"), Action::Paste), + menu_item(fl!("select-all"), Action::SelectAll), MenuTree::new(horizontal_rule(1)), - menu_item(fl!("find"), Message::Find(true)), + menu_item(fl!("find"), Action::Find), ], ), MenuTree::with_children( menu_root(fl!("view")), - vec![menu_item( - fl!("menu-settings"), - Message::ToggleContextPage(ContextPage::Settings), - )], + vec![ + menu_item(fl!("zoom-in"), Action::ZoomIn), + menu_item(fl!("zoom-reset"), Action::ZoomReset), + menu_item(fl!("zoom-out"), Action::ZoomOut), + MenuTree::new(horizontal_rule(1)), + menu_item(fl!("next-tab"), Action::TabNext), + menu_item(fl!("previous-tab"), Action::TabPrev), + MenuTree::new(horizontal_rule(1)), + menu_item(fl!("split-horizontal"), Action::PaneSplitHorizontal), + menu_item(fl!("split-vertical"), Action::PaneSplitVertical), + menu_item(fl!("pane-toggle-maximize"), Action::PaneToggleMaximized), + MenuTree::new(horizontal_rule(1)), + menu_item(fl!("menu-settings"), Action::Settings), + ], ), ]) .item_height(ItemHeight::Dynamic(40))