refactor: about page as a widget

Cargo.toml

@@ -40,7 +40,7 @@ desktop = [
     "process",
     "dep:freedesktop-desktop-entry",
     "dep:mime",
-    "dep:open",
+    "dep:license",
     "dep:shlex",
     "tokio?/io-util",
     "tokio?/net",
@@ -98,8 +98,8 @@ fraction = "0.15.3"
 image = { version = "0.25.1", optional = true }
 lazy_static = "1.4.0"
 libc = { version = "0.2.155", optional = true }
+license = { version = "3.5.1", optional = true }
 mime = { version = "0.3.17", optional = true }
-open = { version = "5.3.0", optional = true }
 palette = "0.7.3"
 rfd = { version = "0.14.0", optional = true }
 rustix = { version = "0.38.34", features = [

examples/about/Cargo.toml

@@ -7,6 +7,7 @@ edition = "2021"
 tracing = "0.1.37"
 tracing-subscriber = "0.3.17"
 tracing-log = "0.2.0"
+open = "5.3.0"
 
 [dependencies.libcosmic]
 path = "../../"

examples/about/src/main.rs

@@ -3,10 +3,10 @@
 
 //! Application API example
 
-use cosmic::app::{about::About, Core, Settings, Task};
+use cosmic::app::{Core, Settings, Task};
 use cosmic::iced::widget::column;
 use cosmic::iced_core::Size;
-use cosmic::widget::{self, nav_bar};
+use cosmic::widget::{self, about::About, nav_bar};
 use cosmic::{executor, iced, ApplicationExt, Element};
 
 /// Runs application with these settings
@@ -27,7 +27,7 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
 #[derive(Clone, Debug)]
 pub enum Message {
     ToggleAbout,
-    Cosmic(cosmic::app::cosmic::Message),
+    Open(String),
 }
 
 /// The [`App`] stores application-specific state.
@@ -64,14 +64,17 @@ impl cosmic::Application for App {
         let nav_model = nav_bar::Model::default();
 
         let about = About::default()
-            .set_application_name("About Demo")
-            .set_application_icon(Self::APP_ID)
-            .set_developer_name("System 76")
-            .set_license_type("GPL-3.0")
-            .set_website("https://system76.com/cosmic")
-            .set_repository_url("https://github.com/pop-os/libcosmic")
-            .set_support_url("https://github.com/pop-os/libcosmic/issues")
-            .set_developers([("Michael Murphy".into(), "mmstick@system76.com".into())]);
+            .name("About Demo")
+            .icon(Self::APP_ID)
+            .version("0.1.0")
+            .author("System 76")
+            .license("GPL-3.0-only")
+            .developers([("Michael Murphy", "mmstick@system76.com")])
+            .links([
+                ("Website", "https://system76.com/cosmic"),
+                ("Repository", "https://github.com/pop-os/libcosmic"),
+                ("Support", "https://github.com/pop-os/libcosmic/issues"),
+            ]);
 
         let mut app = App {
             core,
@@ -100,11 +103,7 @@ impl cosmic::Application for App {
             return None;
         }
 
-        if let Some(abuot_view) = self.about_view() {
-            Some(abuot_view.map(Message::Cosmic))
-        } else {
-            None
-        }
+        Some(widget::about(&self.about, Message::Open))
     }
 
     /// Handle application events here.
@@ -114,17 +113,14 @@ impl cosmic::Application for App {
                 self.core.window.show_context = !self.core.window.show_context;
                 self.core.set_show_context(self.core.window.show_context)
             }
-            Message::Cosmic(message) => {
-                return cosmic::command::message(cosmic::app::Message::Cosmic(message))
-            }
+            Message::Open(url) => match open::that_detached(url) {
+                Ok(_) => (),
+                Err(err) => tracing::error!("Failed to open URL: {err}"),
+            },
         }
         Task::none()
     }
 
-    fn about(&self) -> Option<&About> {
-        Some(&self.about)
-    }
-
     /// Creates a view after each update.
     fn view(&self) -> Element<Self::Message> {
         let centered = cosmic::widget::container(

src/app/cosmic.rs

@@ -78,9 +78,6 @@ pub enum Message {
     /// Tracks updates to window suggested size.
     #[cfg(feature = "applet")]
     SuggestedBounds(Option<iced::Size>),
-    #[cfg(feature = "desktop")]
-    /// Opens the provided URL.
-    OpenUrl(String),
 }
 
 #[derive(Default)]
@@ -664,11 +661,6 @@ impl<T: Application> Cosmic<T> {
                 let core = self.app.core_mut();
                 core.applet.suggested_bounds = b;
             }
-            #[cfg(feature = "desktop")]
-            Message::OpenUrl(url) => match open::that_detached(url) {
-                Ok(_) => (),
-                Err(err) => tracing::error!("{err}"),
-            },
             _ => {}
         }
 

src/app/mod.rs

@@ -6,9 +6,6 @@
 //! Check out our [application](https://github.com/pop-os/libcosmic/tree/master/examples/application)
 //! example in our repository.
 
-#[cfg(feature = "desktop")]
-pub mod about;
-
 pub mod command;
 mod core;
 pub mod cosmic;
@@ -76,7 +73,6 @@ use {
 
 #[cfg(feature = "desktop")]
 use {
-    crate::app::about::About,
     crate::widget,
     iced::{alignment::Vertical, Alignment},
     std::collections::BTreeMap,
@@ -602,90 +598,6 @@ where
         panic!("no view for window {id:?}");
     }
 
-    #[cfg(feature = "desktop")]
-    /// Provides information about the application.
-    fn about(&self) -> Option<&About> {
-        None
-    }
-
-    #[cfg(feature = "desktop")]
-    /// Constructs the view for the about section.
-    fn about_view<'a>(&'a self) -> Option<Element<'a, crate::app::cosmic::Message>> {
-        let about = self.about()?;
-
-        let spacing = crate::theme::active().cosmic().spacing;
-
-        let section = |list: &'a BTreeMap<String, String>, title: &'a str| {
-            if list.is_empty() {
-                None
-            } else {
-                let developers: Vec<Element<crate::app::cosmic::Message>> = list
-                    .into_iter()
-                    .map(|(name, url)| {
-                        widget::button::custom(
-                            widget::row()
-                                .push(widget::text(name))
-                                .push(horizontal_space())
-                                .push(crate::widget::icon::from_name("link-symbolic").icon())
-                                .padding(spacing.space_xxs)
-                                .align_y(Vertical::Center),
-                        )
-                        .class(crate::theme::Button::Text)
-                        .on_press(crate::app::cosmic::Message::OpenUrl(url.clone()))
-                        .width(Length::Fill)
-                        .into()
-                    })
-                    .collect();
-                Some(widget::settings::section().title(title).extend(developers))
-            }
-        };
-
-        let application_name = about.application_name.as_ref().map(widget::text::title3);
-        let application_icon = about
-            .application_icon
-            .as_ref()
-            .map(|icon| crate::desktop::IconSource::Name(icon.clone()).as_cosmic_icon());
-
-        let links_section = section(&about.links, "Links");
-        let developers_section = section(&about.developers, "Developers");
-        let designers_section = section(&about.designers, "Designers");
-        let artists_section = section(&about.artists, "Artists");
-        let translators_section = section(&about.translators, "Translators");
-        let documenters_section = section(&about.documenters, "Documenters");
-
-        let developer_name = about.developer_name.as_ref().map(widget::text);
-        let version = about.version.as_ref().map(widget::button::standard);
-        let license = about.license_type.as_ref().map(widget::button::standard);
-        let copyright = about.copyright.as_ref().map(widget::text::body);
-        let comments = about.comments.as_ref().map(widget::text::body);
-
-        let about = widget::scrollable(
-            widget::column()
-                .push_maybe(application_icon)
-                .push_maybe(application_name)
-                .push_maybe(developer_name)
-                .push(
-                    widget::row()
-                        .push_maybe(version)
-                        .push_maybe(license)
-                        .spacing(spacing.space_xs),
-                )
-                .push_maybe(links_section)
-                .push_maybe(developers_section)
-                .push_maybe(designers_section)
-                .push_maybe(artists_section)
-                .push_maybe(translators_section)
-                .push_maybe(documenters_section)
-                .push_maybe(comments)
-                .push_maybe(copyright)
-                .align_x(Alignment::Center)
-                .spacing(spacing.space_xs)
-                .width(Length::Fill),
-        )
-        .into();
-        Some(about)
-    }
-
     /// Overrides the default style for applications
     fn style(&self) -> Option<iced_runtime::Appearance> {
         None