[WIP] status-area: Handle menuless items properly

cosmic-applet-status-area/src/components/app.rs

@@ -26,6 +26,7 @@ pub enum Msg {
     TogglePopup(usize),
     Hovered(usize),
     Surface(surface::Action),
+    RightPressed(usize),
 }
 
 #[derive(Default)]
@@ -188,6 +189,10 @@ impl cosmic::Application for App {
                 }
                 Task::none()
             }
+            Msg::RightPressed(id) => {
+                self.menus[&id].ctx_menu_activate();
+                Task::none()
+            }
             Msg::Hovered(id) => {
                 let mut cmds = Vec::new();
                 if let Some(old_id) = self.open_menu.take() {
@@ -261,6 +266,7 @@ impl cosmic::Application for App {
                 .on_press_down(Msg::TogglePopup(*id)),
             )
             .on_enter(Msg::Hovered(*id))
+            .on_right_press(Msg::RightPressed(*id))
             .into()
         });
         self.core

cosmic-applet-status-area/src/components/status_menu.rs

@@ -71,8 +71,11 @@ impl State {
 
                 iced::Task::none()
             }
-            Msg::Click(id, is_submenu) => {
-                let menu_proxy = self.item.menu_proxy().clone();
+            Msg::Click(id, is_submenu) => 'block: {
+                let Some(menu_proxy) = self.item.menu_proxy().cloned() else {
+                    tracing::error!("Msg::click on item without menu_proxy");
+                    break 'block iced::Task::none();
+                };
                 tokio::spawn(async move {
                     let _ = menu_proxy.event(id, "clicked", &0.into(), 0).await;
                 });
@@ -118,15 +121,48 @@ impl State {
     }
 
     pub fn opened(&self) {
-        let menu_proxy = self.item.menu_proxy().clone();
+        let Some(menu_proxy) = self.item.menu_proxy().cloned() else {
+            let item_proxy = self.item.item_proxy().clone();
+            tokio::spawn(async move {
+                if let Err(e) = item_proxy.activate(0, 0).await {
+                    tracing::error!("Error on Activate: {e}");
+                }
+            });
+            return;
+        };
+        let item_proxy = self.item.item_proxy().clone();
+        tokio::spawn(async move {
+            let is_menu = match item_proxy.item_is_menu().await {
+                Ok(is_menu) => is_menu,
+                Err(e) => {
+                    tracing::error!("Error on ItemIsMenu: {e}");
+                    false
+                }
+            };
+            if is_menu {
+                let _ = menu_proxy.event(0, "opened", &0i32.into(), 0).await;
+                let _ = menu_proxy.about_to_show(0).await;
+            } else {
+                if let Err(e) = item_proxy.activate(0, 0).await {
+                    tracing::error!("Error on Activate: {e}");
+                }
+            }
+        });
+    }
+
+    pub fn ctx_menu_activate(&self) {
+        let item_proxy = self.item.item_proxy().clone();
         tokio::spawn(async move {
-            let _ = menu_proxy.event(0, "opened", &0i32.into(), 0).await;
-            let _ = menu_proxy.about_to_show(0).await;
+            if let Err(e) = item_proxy.context_menu(1920, 0).await {
+                tracing::error!("Error on ContextMenu: {e}");
+            }
         });
     }
 
     pub fn closed(&self) {
-        let menu_proxy = self.item.menu_proxy().clone();
+        let Some(menu_proxy) = self.item.menu_proxy().cloned() else {
+            return;
+        };
         tokio::spawn(async move {
             let _ = menu_proxy.event(0, "closed", &0i32.into(), 0).await;
         });

cosmic-applet-status-area/src/subscriptions/status_notifier_item.rs

@@ -12,7 +12,7 @@ use zbus::zvariant::{self, OwnedValue};
 pub struct StatusNotifierItem {
     name: String,
     item_proxy: StatusNotifierItemProxy<'static>,
-    menu_proxy: DBusMenuProxy<'static>,
+    menu_proxy: Option<DBusMenuProxy<'static>>,
 }
 
 #[derive(Clone, Debug, zvariant::Value)]
@@ -44,12 +44,19 @@ impl StatusNotifierItem {
             .build()
             .await?;
 
-        let menu_path = item_proxy.menu().await?;
-        let menu_proxy = DBusMenuProxy::builder(connection)
-            .destination(dest.to_string())?
-            .path(menu_path)?
-            .build()
-            .await?;
+        let menu_proxy = match item_proxy.menu().await {
+            Ok(menu_path) => Some(
+                DBusMenuProxy::builder(connection)
+                    .destination(dest.to_string())?
+                    .path(menu_path)?
+                    .build()
+                    .await?,
+            ),
+            Err(e) => {
+                eprintln!("Error: {e}");
+                None
+            }
+        };
 
         Ok(Self {
             name,
@@ -64,7 +71,9 @@ impl StatusNotifierItem {
 
     // TODO: Only fetch changed part of layout, if that's any faster
     pub fn layout_subscription(&self) -> iced::Subscription<Result<Layout, String>> {
-        let menu_proxy = self.menu_proxy.clone();
+        let Some(menu_proxy) = self.menu_proxy.clone() else {
+            return iced::Subscription::none();
+        };
         Subscription::run_with_id(
             format!("status-notifier-item-layout-{}", &self.name),
             async move {
@@ -109,8 +118,12 @@ impl StatusNotifierItem {
         )
     }
 
-    pub fn menu_proxy(&self) -> &DBusMenuProxy<'static> {
-        &self.menu_proxy
+    pub fn menu_proxy(&self) -> Option<&DBusMenuProxy<'static>> {
+        self.menu_proxy.as_ref()
+    }
+
+    pub fn item_proxy(&self) -> &StatusNotifierItemProxy<'static> {
+        &self.item_proxy
     }
 }
 
@@ -135,6 +148,10 @@ trait StatusNotifierItem {
 
     #[zbus(signal)]
     fn new_icon(&self) -> zbus::Result<()>;
+
+    fn activate(&self, x: i32, y: i32) -> zbus::Result<()>;
+    fn context_menu(&self, x: i32, y: i32) -> zbus::Result<()>;
+    fn item_is_menu(&self) -> zbus::Result<bool>;
 }
 
 #[derive(Clone, Debug)]