From b3fc1aef9142ea6961896e20db4ada2f0225100a Mon Sep 17 00:00:00 2001 From: Vladislav Nepogodin Date: Tue, 6 Aug 2024 20:34:41 +0400 Subject: [PATCH] =?UTF-8?q?=F0=9F=9A=A7=20pages:=20refactor=20kwin=20debug?= =?UTF-8?q?=20window=20into=20helper=20function?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/pages.rs | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/src/pages.rs b/src/pages.rs index 10caf29..a38bfe9 100644 --- a/src/pages.rs +++ b/src/pages.rs @@ -233,6 +233,13 @@ fn get_nm_connections() -> Vec { connections.split('\n').filter(|x| !x.is_empty()).map(String::from).collect::>() } +fn launch_kwin_debug_window() { + let _ = Exec::cmd("qdbus6") + .args(&["org.kde.KWin", "/KWin", "org.kde.KWin.showDebugConsole"]) + .join() + .unwrap(); +} + fn create_fixes_section(builder: &Builder) -> gtk::Box { let topbox = gtk::Box::new(gtk::Orientation::Vertical, 2); let button_box_f = gtk::Box::new(gtk::Orientation::Horizontal, 10); @@ -448,10 +455,8 @@ fn create_fixes_section(builder: &Builder) -> gtk::Box { kwinw_debug_btn.connect_clicked(move |_| { // Spawn child process in separate thread. std::thread::spawn(move || { - let _ = Exec::cmd("qdbus6") - .args(&["org.kde.KWin", "/KWin", "org.kde.KWin.showDebugConsole"]) - .join() - .unwrap(); + // do we even need to start that in separate thread. should be fine without + launch_kwin_debug_window(); }); }); button_box_frth.pack_end(&kwinw_debug_btn, true, true, 2);