[pdm-devel] [PATCH datacenter-manager 04/15] ui: dashboard: remote panel: make wizard menu optional

[Dominik Csapak]

if the pve/pbs wizard callbacks are None, don't show the menu

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
 ui/src/dashboard/mod.rs          | 12 ++++++---
 ui/src/dashboard/remote_panel.rs | 43 ++++++++++++++++++--------------
 2 files changed, 32 insertions(+), 23 deletions(-)

diff --git a/ui/src/dashboard/mod.rs b/ui/src/dashboard/mod.rs
index 2fe4d7fa..0441440a 100644
--- a/ui/src/dashboard/mod.rs
+++ b/ui/src/dashboard/mod.rs
@@ -423,10 +423,14 @@ impl Component for PdmDashboard {
                     .with_child(
                         create_remote_panel(
                             self.status.clone(),
-                            ctx.link()
-                                .callback(|_| Msg::CreateWizard(Some(RemoteType::Pve))),
-                            ctx.link()
-                                .callback(|_| Msg::CreateWizard(Some(RemoteType::Pbs))),
+                            Some(
+                                ctx.link()
+                                    .callback(|_| Msg::CreateWizard(Some(RemoteType::Pve))),
+                            ),
+                            Some(
+                                ctx.link()
+                                    .callback(|_| Msg::CreateWizard(Some(RemoteType::Pbs))),
+                            ),
                         )
                         .flex(1.0)
                         .width(300)
diff --git a/ui/src/dashboard/remote_panel.rs b/ui/src/dashboard/remote_panel.rs
index 27eebac2..747f9b8d 100644
--- a/ui/src/dashboard/remote_panel.rs
+++ b/ui/src/dashboard/remote_panel.rs
@@ -117,26 +117,31 @@ fn create_search_term(failure: bool) -> Search {
 
 pub fn create_remote_panel(
     status: Option<ResourcesStatus>,
-    on_pve_wizard: impl IntoEventCallback<MenuEvent>,
-    on_pbs_wizard: impl IntoEventCallback<MenuEvent>,
+    on_pve_wizard: Option<impl IntoEventCallback<MenuEvent>>,
+    on_pbs_wizard: Option<impl IntoEventCallback<MenuEvent>>,
 ) -> Panel {
-    Panel::new()
+    let mut panel = Panel::new()
         .title(create_title_with_icon("server", tr!("Remotes")))
         .border(true)
-        .with_tool(
-            MenuButton::new(tr!("Add")).show_arrow(true).menu(
-                Menu::new()
-                    .with_item(
-                        MenuItem::new("Proxmox VE")
-                            .icon_class("fa fa-building")
-                            .on_select(on_pve_wizard),
-                    )
-                    .with_item(
-                        MenuItem::new("Proxmox Backup Server")
-                            .icon_class("fa fa-floppy-o")
-                            .on_select(on_pbs_wizard),
-                    ),
-            ),
-        )
-        .with_child(RemotePanel::new(status))
+        .with_child(RemotePanel::new(status));
+
+    if on_pve_wizard.is_some() || on_pbs_wizard.is_some() {
+        let mut menu = Menu::new();
+        if let Some(on_pve_wizard) = on_pve_wizard {
+            menu.add_item(
+                MenuItem::new("Proxmox VE")
+                    .icon_class("fa fa-building")
+                    .on_select(on_pve_wizard),
+            );
+        }
+        if let Some(on_pbs_wizard) = on_pbs_wizard {
+            menu.add_item(
+                MenuItem::new("Proxmox Backup Server")
+                    .icon_class("fa fa-floppy-o")
+                    .on_select(on_pbs_wizard),
+            );
+        }
+        panel.add_tool(MenuButton::new(tr!("Add")).show_arrow(true).menu(menu));
+    }
+    panel
 }
-- 
2.47.3