Merge pull request #162 from edfloreshz/master

refactor(menu): update menu declaration.

Cargo.toml

@@ -58,4 +58,4 @@ wgpu = ["libcosmic/wgpu", "cosmic-files/wgpu"]
 
 [profile.release-with-debug]
 inherits = "release"
-debug = true
+debug = true

src/key_bind.rs

@@ -1,49 +1,9 @@
-use cosmic::{
+use cosmic::widget::menu::key_bind::{KeyBind, Modifier};
-    iced::keyboard::{Key, Modifiers},
+use cosmic::{iced::keyboard::Key, iced_core::keyboard::key::Named};
-    iced_core::keyboard::key::Named,
+use std::collections::HashMap;
-};
-use serde::{Deserialize, Serialize};
-use std::{collections::HashMap, fmt};
 
 use crate::Action;
 
-#[derive(Clone, Copy, Debug, Deserialize, Eq, Hash, Ord, PartialEq, PartialOrd, Serialize)]
-pub enum Modifier {
-    Super,
-    Ctrl,
-    Alt,
-    Shift,
-}
-
-#[derive(Clone, Debug, Deserialize, Eq, Hash, Ord, PartialEq, PartialOrd, Serialize)]
-pub struct KeyBind {
-    pub modifiers: Vec<Modifier>,
-    pub key: Key,
-}
-
-impl KeyBind {
-    pub fn matches(&self, modifiers: Modifiers, key: &Key) -> bool {
-        key == &self.key
-            && modifiers.logo() == self.modifiers.contains(&Modifier::Super)
-            && modifiers.control() == self.modifiers.contains(&Modifier::Ctrl)
-            && modifiers.alt() == self.modifiers.contains(&Modifier::Alt)
-            && modifiers.shift() == self.modifiers.contains(&Modifier::Shift)
-    }
-}
-
-impl fmt::Display for KeyBind {
-    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
-        for modifier in self.modifiers.iter() {
-            write!(f, "{:?} + ", modifier)?;
-        }
-        match &self.key {
-            Key::Character(c) => write!(f, "{}", c.to_uppercase()),
-            Key::Named(named) => write!(f, "{:?}", named),
-            other => write!(f, "{:?}", other),
-        }
-    }
-}
-
 //TODO: load from config
 pub fn key_binds() -> HashMap<KeyBind, Action> {
     let mut key_binds = HashMap::new();

src/main.rs

@@ -1,5 +1,8 @@
 // SPDX-License-Identifier: GPL-3.0-only
 
+use cosmic::widget::menu::action::MenuAction;
+use cosmic::widget::menu::key_bind::KeyBind;
+use cosmic::widget::segmented_button::Entity;
 use cosmic::{
     app::{message, Command, Core, Settings},
     cosmic_config::{self, CosmicConfigEntry},
@@ -44,7 +47,7 @@ mod git;
 use icon_cache::IconCache;
 mod icon_cache;
 
-use key_bind::{key_binds, KeyBind};
+use key_bind::key_binds;
 mod key_bind;
 
 use line_number::LineNumberCache;
@@ -224,8 +227,9 @@ pub enum Action {
     Undo,
 }
 
-impl Action {
+impl MenuAction for Action {
-    pub fn message(&self) -> Message {
+    type Message = Message;
+    fn message(&self, _entity: Option<Entity>) -> Message {
         match self {
             Self::Todo => Message::Todo,
             Self::About => Message::ToggleContextPage(ContextPage::About),
@@ -1608,7 +1612,7 @@ impl Application for App {
             Message::Key(modifiers, key) => {
                 for (key_bind, action) in self.key_binds.iter() {
                     if key_bind.matches(modifiers, &key) {
-                        return self.update(action.message());
+                        return self.update(action.message(None));
                     }
                 }
             }
@@ -2100,7 +2104,7 @@ impl Application for App {
                     // Close context menu
                     tab.context_menu = None;
                     // Run action's message
-                    return self.update(action.message());
+                    return self.update(action.message(None));
                 }
             }
             Message::TabContextMenu(entity, position_opt) => {

src/menu.rs

@@ -1,13 +1,11 @@
 // SPDX-License-Identifier: GPL-3.0-only
 
+use cosmic::widget::menu::key_bind::KeyBind;
+use cosmic::widget::menu::menu_tree::{menu_items, menu_root, MenuItem};
 use cosmic::{
-    //TODO: export in cosmic::widget
+    iced::{widget::column, widget::horizontal_rule, Alignment, Background, Length},
-    iced::{
-        widget::{column, horizontal_rule},
-        Alignment, Background, Length,
-    },
     iced_core::Border,
-    theme,
+    menu_button, theme,
     widget::{
         self, horizontal_space,
         menu::{ItemHeight, ItemWidth, MenuBar, MenuTree},
@@ -17,22 +15,7 @@ use cosmic::{
 };
 use std::{collections::HashMap, path::PathBuf};
 
-use crate::{fl, icon_cache_get, Action, Config, ConfigState, KeyBind, Message};
+use crate::{fl, Action, Config, ConfigState, Message};
-
-macro_rules! menu_button {
-    ($($x:expr),+ $(,)?) => (
-        widget::button(
-            widget::Row::with_children(
-                vec![$(Element::from($x)),+]
-            )
-            .align_items(Alignment::Center)
-        )
-        .height(Length::Fixed(32.0))
-        .padding([4, 16])
-        .width(Length::Fill)
-        .style(theme::Button::MenuItem)
-    );
-}
 
 pub fn context_menu<'a>(
     key_binds: &HashMap<KeyBind, Action>,
@@ -91,61 +74,8 @@ pub fn menu_bar<'a>(
     projects: &Vec<(String, PathBuf)>,
 ) -> Element<'a, Message> {
     //TODO: port to libcosmic
-    let menu_root = |label| {
-        widget::button(widget::text(label))
-            .padding([4, 12])
-            .style(theme::Button::MenuRoot)
-    };
-
-    let menu_folder =
-        |label| menu_button!(widget::text(label), horizontal_space(Length::Fill), ">");
-
-    let find_key = |action: &Action| -> String {
-        for (key_bind, key_action) in key_binds.iter() {
-            if action == key_action {
-                return key_bind.to_string();
-            }
-        }
-        if action == &Action::Todo {
-            return fl!("todo");
-        }
-        String::new()
-    };
-
-    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(action.message()),
-        )
-    };
-
-    //TODO: support key lookup?
-    let menu_checkbox = |label, value, action| {
-        let check: Element<_> = if value {
-            icon_cache_get("object-select-symbolic", 16).into()
-        } else {
-            widget::Space::with_width(Length::Fixed(16.0)).into()
-        };
-        let key = find_key(&action);
-        MenuTree::new(
-            menu_button!(
-                check,
-                widget::Space::with_width(Length::Fixed(8.0)),
-                widget::text(label),
-                horizontal_space(Length::Fill),
-                widget::text(key)
-            )
-            .on_press(action.message()),
-        )
-    };
-
     let menu_tab_width = |tab_width: u16| {
-        menu_checkbox(
+        MenuItem::CheckBox(
             fl!("tab-width", tab_width = tab_width),
             config.tab_width == tab_width,
             Action::TabWidth(tab_width),
@@ -164,114 +94,132 @@ pub fn menu_bar<'a>(
 
     let mut recent_files = Vec::with_capacity(config_state.recent_files.len());
     for (i, path) in config_state.recent_files.iter().enumerate() {
-        recent_files.push(menu_item(format_path(path), Action::OpenRecentFile(i)));
+        recent_files.push(MenuItem::Button(
+            format_path(path),
+            Action::OpenRecentFile(i),
+        ));
     }
 
     let mut recent_projects = Vec::with_capacity(config_state.recent_projects.len());
     for (i, path) in config_state.recent_projects.iter().enumerate() {
-        recent_projects.push(menu_item(format_path(path), Action::OpenRecentProject(i)));
+        recent_projects.push(MenuItem::Button(
+            format_path(path),
+            Action::OpenRecentProject(i),
+        ));
     }
 
     let mut close_projects = Vec::with_capacity(projects.len());
     for (project_i, (name, _path)) in projects.iter().enumerate() {
-        close_projects.push(menu_item(name.clone(), Action::CloseProject(project_i)));
+        close_projects.push(MenuItem::Button(
+            name.clone(),
+            Action::CloseProject(project_i),
+        ));
     }
 
     MenuBar::new(vec![
         MenuTree::with_children(
             menu_root(fl!("file")),
-            vec![
+            menu_items(
-                menu_item(fl!("new-file"), Action::NewFile),
+                key_binds,
-                menu_item(fl!("new-window"), Action::NewWindow),
+                vec![
-                MenuTree::new(horizontal_rule(1)),
+                    MenuItem::Button(fl!("new-file"), Action::NewFile),
-                menu_item(fl!("open-file"), Action::OpenFileDialog),
+                    MenuItem::Button(fl!("new-window"), Action::NewWindow),
-                MenuTree::with_children(menu_folder(fl!("open-recent-file")), recent_files),
+                    MenuItem::Divider,
-                menu_item(fl!("close-file"), Action::CloseFile),
+                    MenuItem::Button(fl!("open-file"), Action::OpenFileDialog),
-                MenuTree::new(horizontal_rule(1)),
+                    MenuItem::Folder(fl!("open-recent-file"), recent_files),
-                menu_item(fl!("menu-open-project"), Action::OpenProjectDialog),
+                    MenuItem::Button(fl!("close-file"), Action::CloseFile),
-                MenuTree::with_children(menu_folder(fl!("open-recent-project")), recent_projects),
+                    MenuItem::Divider,
-                MenuTree::with_children(menu_folder(fl!("close-project")), close_projects),
+                    MenuItem::Button(fl!("menu-open-project"), Action::OpenProjectDialog),
-                MenuTree::new(horizontal_rule(1)),
+                    MenuItem::Folder(fl!("open-recent-project"), recent_projects),
-                menu_item(fl!("save"), Action::Save),
+                    MenuItem::Folder(fl!("close-project"), close_projects),
-                menu_item(fl!("save-as"), Action::SaveAsDialog),
+                    MenuItem::Divider,
-                MenuTree::new(horizontal_rule(1)),
+                    MenuItem::Button(fl!("save"), Action::Save),
-                menu_item(fl!("revert-all-changes"), Action::Todo),
+                    MenuItem::Button(fl!("save-as"), Action::SaveAsDialog),
-                MenuTree::new(horizontal_rule(1)),
+                    MenuItem::Divider,
-                menu_item(
+                    MenuItem::Button(fl!("revert-all-changes"), Action::Todo),
-                    fl!("menu-document-statistics"),
+                    MenuItem::Divider,
-                    Action::ToggleDocumentStatistics,
+                    MenuItem::Button(
-                ),
+                        fl!("menu-document-statistics"),
-                //TODO menu_item(fl!("document-type"), Action::Todo),
+                        Action::ToggleDocumentStatistics,
-                //TODO menu_item(fl!("encoding"), Action::Todo),
+                    ),
-                menu_item(fl!("menu-git-management"), Action::ToggleGitManagement),
+                    //TODO MenuItem::Button(fl!("document-type"), Action::Todo),
-                //TODO menu_item(fl!("print"), Action::Todo),
+                    //TODO MenuItem::Button(fl!("encoding"), Action::Todo),
-                MenuTree::new(horizontal_rule(1)),
+                    MenuItem::Button(fl!("menu-git-management"), Action::ToggleGitManagement),
-                menu_item(fl!("quit"), Action::Quit),
+                    //TODO MenuItem::Button(fl!("print"), Action::Todo),
-            ],
+                    MenuItem::Divider,
+                    MenuItem::Button(fl!("quit"), Action::Quit),
+                ],
+            ),
         ),
         MenuTree::with_children(
             menu_root(fl!("edit")),
-            vec![
+            menu_items(
-                menu_item(fl!("undo"), Action::Undo),
+                key_binds,
-                menu_item(fl!("redo"), Action::Redo),
+                vec![
-                MenuTree::new(horizontal_rule(1)),
+                    MenuItem::Button(fl!("undo"), Action::Undo),
-                menu_item(fl!("cut"), Action::Cut),
+                    MenuItem::Button(fl!("redo"), Action::Redo),
-                menu_item(fl!("copy"), Action::Copy),
+                    MenuItem::Divider,
-                menu_item(fl!("paste"), Action::Paste),
+                    MenuItem::Button(fl!("cut"), Action::Cut),
-                menu_item(fl!("select-all"), Action::SelectAll),
+                    MenuItem::Button(fl!("copy"), Action::Copy),
-                MenuTree::new(horizontal_rule(1)),
+                    MenuItem::Button(fl!("paste"), Action::Paste),
-                menu_item(fl!("find"), Action::Find),
+                    MenuItem::Button(fl!("select-all"), Action::SelectAll),
-                menu_item(fl!("replace"), Action::FindAndReplace),
+                    MenuItem::Divider,
-                menu_item(fl!("find-in-project"), Action::ToggleProjectSearch),
+                    MenuItem::Button(fl!("find"), Action::Find),
-                MenuTree::new(horizontal_rule(1)),
+                    MenuItem::Button(fl!("replace"), Action::FindAndReplace),
-                menu_item(fl!("spell-check"), Action::Todo),
+                    MenuItem::Button(fl!("find-in-project"), Action::ToggleProjectSearch),
-            ],
+                    MenuItem::Divider,
+                    MenuItem::Button(fl!("spell-check"), Action::Todo),
+                ],
+            ),
         ),
         MenuTree::with_children(
             menu_root(fl!("view")),
-            vec![
+            menu_items(
-                MenuTree::with_children(
+                key_binds,
-                    menu_folder(fl!("indentation")),
+                vec![
-                    vec![
+                    MenuItem::Folder(
-                        menu_checkbox(
+                        fl!("indentation"),
-                            fl!("automatic-indentation"),
+                        vec![
-                            config.auto_indent,
+                            MenuItem::CheckBox(
-                            Action::ToggleAutoIndent,
+                                fl!("automatic-indentation"),
-                        ),
+                                config.auto_indent,
-                        MenuTree::new(horizontal_rule(1)),
+                                Action::ToggleAutoIndent,
-                        menu_tab_width(1),
+                            ),
-                        menu_tab_width(2),
+                            MenuItem::Divider,
-                        menu_tab_width(3),
+                            menu_tab_width(1),
-                        menu_tab_width(4),
+                            menu_tab_width(2),
-                        menu_tab_width(5),
+                            menu_tab_width(3),
-                        menu_tab_width(6),
+                            menu_tab_width(4),
-                        menu_tab_width(7),
+                            menu_tab_width(5),
-                        menu_tab_width(8),
+                            menu_tab_width(6),
-                        //TODO MenuTree::new(horizontal_rule(1)),
+                            menu_tab_width(7),
-                        //TODO menu_item(fl!("convert-indentation-to-spaces"), Action::Todo),
+                            menu_tab_width(8),
-                        //TODO menu_item(fl!("convert-indentation-to-tabs"), Action::Todo),
+                            //TODO MenuItem::Divider,
-                    ],
+                            //TODO MenuItem::Button(fl!("convert-indentation-to-spaces"), Action::Todo),
-                ),
+                            //TODO MenuItem::Button(fl!("convert-indentation-to-tabs"), Action::Todo),
-                MenuTree::new(horizontal_rule(1)),
+                        ],
-                menu_checkbox(fl!("word-wrap"), config.word_wrap, Action::ToggleWordWrap),
+                    ),
-                menu_checkbox(
+                    MenuItem::Divider,
-                    fl!("show-line-numbers"),
+                    MenuItem::CheckBox(fl!("word-wrap"), config.word_wrap, Action::ToggleWordWrap),
-                    config.line_numbers,
+                    MenuItem::CheckBox(
-                    Action::ToggleLineNumbers,
+                        fl!("show-line-numbers"),
-                ),
+                        config.line_numbers,
-                menu_checkbox(
+                        Action::ToggleLineNumbers,
-                    fl!("highlight-current-line"),
+                    ),
-                    config.highlight_current_line,
+                    MenuItem::CheckBox(
-                    Action::ToggleHighlightCurrentLine,
+                        fl!("highlight-current-line"),
-                ),
+                        config.highlight_current_line,
-                //TODO: menu_item(fl!("syntax-highlighting"), Action::Todo),
+                        Action::ToggleHighlightCurrentLine,
-                MenuTree::new(horizontal_rule(1)),
+                    ),
-                menu_item(fl!("menu-settings"), Action::ToggleSettingsPage),
+                    //TODO: MenuItem::CheckBox(fl!("syntax-highlighting"), Action::Todo),
-                //TODO MenuTree::new(horizontal_rule(1)),
+                    MenuItem::Divider,
-                //TODO menu_item(fl!("menu-keyboard-shortcuts"), Action::Todo),
+                    MenuItem::Button(fl!("menu-settings"), Action::ToggleSettingsPage),
-                MenuTree::new(horizontal_rule(1)),
+                    //TODO MenuItem::Divider,
-                menu_item(fl!("menu-about"), Action::About),
+                    //TODO MenuItem::Button(fl!("menu-keyboard-shortcuts"), Action::Todo),
-            ],
+                    MenuItem::Divider,
+                    MenuItem::Button(fl!("menu-about"), Action::About),
+                ],
+            ),
         ),
     ])
     .item_height(ItemHeight::Dynamic(40))