feat(preferences): default --no-entry on and clarify app-list visibility

Author: ranfdev

data/com.ranfdev.DistroShelf.gschema.xml

@@ -21,5 +21,10 @@
 			<summary>Distrobox executable source</summary>
 			<description>The source of the distrobox executable. Can be 'host' or 'bundled'.</description>
 		</key>
+		<key name="distrobox-create-no-entry" type="b">
+			<default>true</default>
+			<summary>Use --no-entry when creating a distrobox</summary>
+			<description>When enabled, distrobox create commands include --no-entry by default.</description>
+		</key>
 	</schema>
 </schemalist>

src/backends/distrobox/distrobox.rs

@@ -241,6 +241,7 @@ pub struct CreateArgs {
     pub init: bool,
     pub nvidia: bool,
     pub root: bool,
+    pub no_entry: bool,
     pub hostname: Option<String>,
     pub home_path: Option<String>,
     pub image: String,
@@ -1024,6 +1025,9 @@ impl Distrobox {
         if args.root {
             cmd.arg("--root");
         }
+        if args.no_entry {
+            cmd.arg("--no-entry");
+        }
         if args.nvidia {
             cmd.arg("--nvidia");
         }
@@ -1377,6 +1381,24 @@ Categories=Utility;Security;";
         );
         Ok(())
     }
+
+    #[test]
+    fn create_with_no_entry() -> Result<(), Error> {
+        let db = Distrobox::new(CommandRunner::new_null(), default_cmd_factory());
+        let output_tracker = db.cmd_runner.output_tracker();
+        let args = CreateArgs {
+            image: "docker.io/library/ubuntu:latest".into(),
+            no_entry: true,
+            ..Default::default()
+        };
+
+        smol::block_on(db.create(args))?;
+
+        let command = output_tracker.items()[0].command().unwrap().to_string();
+        assert!(command.contains(" --no-entry"));
+        Ok(())
+    }
+
     #[test]
     fn assemble() -> Result<(), Error> {
         let db = Distrobox::new(CommandRunner::new_null(), default_cmd_factory());

src/dialogs/create_distrobox_dialog.rs

@@ -956,6 +956,10 @@ impl CreateDistroboxDialog {
             init: imp.init_row.is_active(),
             hostname,
             root: imp.root_row.is_active(),
+            no_entry: self
+                .root_store()
+                .settings()
+                .boolean("distrobox-create-no-entry"),
             volumes,
         };
 

src/dialogs/preferences_dialog.rs

@@ -166,6 +166,20 @@ mod imp {
 
             distrobox_group.add(&distrobox_source_row);
 
+            let no_entry_row = adw::SwitchRow::new();
+            no_entry_row.set_title(&gettext("Use --no-entry for new containers"));
+            no_entry_row.set_subtitle(&gettext(
+                "No .desktop app entry is created, so it won't appear in your app list.",
+            ));
+            no_entry_row.set_active(settings.boolean("distrobox-create-no-entry"));
+
+            let settings_for_no_entry = settings.clone();
+            no_entry_row.connect_active_notify(move |row| {
+                let _ = settings_for_no_entry.set_boolean("distrobox-create-no-entry", row.is_active());
+            });
+
+            distrobox_group.add(&no_entry_row);
+
             // Add version row
             let version_row = adw::ActionRow::new();
             version_row.set_title(&gettext("Distrobox Version"));