fix(sound): use popup dropdown widget

Author: mmstick

cosmic-settings/src/pages/sound/device_profiles.rs

@@ -1,16 +1,15 @@
 // Copyright 2025 System76 <[email protected]>
 // SPDX-License-Identifier: GPL-3.0-only
 
-use cosmic::widget;
+use cosmic::{Apply, widget};
 use cosmic_settings_page::{self as page, Section, section};
 use cosmic_settings_sound_subscription::{self as subscription, pipewire};
 use itertools::Itertools;
 use slotmap::SlotMap;
 
 #[derive(Clone, Debug)]
 pub enum Message {
-    /// Messages handled by the sound module in cosmic-settings-subscriptions
-    Subscription(subscription::Message),
+    Surface(cosmic::surface::Action),
 }
 
 impl From<Message> for crate::pages::Message {
@@ -59,8 +58,12 @@ impl page::Page<crate::pages::Message> for Page {
 }
 
 impl Page {
-    pub fn update(&mut self, _message: Message) -> cosmic::Task<crate::app::Message> {
-        cosmic::Task::none()
+    pub fn update(&mut self, message: Message) -> cosmic::Task<crate::app::Message> {
+        match message {
+            Message::Surface(a) => {
+                return cosmic::Task::done(crate::app::Message::Surface(a));
+            }
+        }
     }
 }
 
@@ -75,15 +78,15 @@ pub fn view() -> Section<crate::pages::Message> {
             .model
             .device_profiles
             .iter()
-            .filter_map(|(object_id, profiles)| {
-                let name = sound_page.model.device_names.get(object_id)?.as_str();
+            .filter_map(|(device_id, profiles)| {
+                let name = sound_page.model.device_names.get(device_id)?.as_str();
 
                 // TODO: cache
                 let active_profile =
                     sound_page
                         .model
                         .active_profiles
-                        .get(object_id)
+                        .get(device_id)
                         .and_then(|profile| {
                             profiles
                                 .iter()
@@ -103,10 +106,17 @@ pub fn view() -> Section<crate::pages::Message> {
                     })
                     .map(|p| p.description.clone());
 
-                let dropdown =
-                    widget::dropdown(Vec::from_iter(profiles), active_profile, move |pos| {
-                        super::Message::SetProfile(object_id, pos).into()
-                    });
+                let dropdown = widget::dropdown::popup_dropdown(
+                    Vec::from_iter(profiles),
+                    active_profile,
+                    move |id| super::Message::SetProfile(device_id, id),
+                    cosmic::iced::window::Id::RESERVED,
+                    super::Message::Surface,
+                    crate::Message::from,
+                )
+                .apply(cosmic::Element::from)
+                .map(crate::pages::Message::from);
+
                 Some((
                     name,
                     widget::settings::item::builder(name).control(dropdown),