fix(linux): Port to GTK4

Closes tauri-apps#1051

Author: conradhale

Cargo.lock

@@ -45,7 +45,7 @@ rwh_04 = { package = "raw-window-handle", version = "0.4", optional = true }
 rwh_05 = { package = "raw-window-handle", version = "0.5", features = [ "std" ], optional = true }
 rwh_06 = { package = "raw-window-handle", version = "0.6", features = [ "std" ], optional = true }
 bitflags = "2"
-crossbeam-channel = "0.5"
+async-channel = "2.3"
 url = "2"
 dpi = "0.1"
 
@@ -146,9 +146,9 @@ dispatch = "0.2"
 scopeguard = "1.2"
 
 [target."cfg(any(target_os = \"linux\", target_os = \"dragonfly\", target_os = \"freebsd\", target_os = \"openbsd\", target_os = \"netbsd\"))".dependencies]
-gtk = "0.18"
-gdkx11-sys = "0.18"
-gdkwayland-sys = "0.18.0"
+gtk = { package = "gtk4", version = "0.9", features = ["v4_6"] }
+gdk-x11-sys = { package = "gdk4-x11-sys", version = "0.9" }
+gdk-wayland-sys = { package = "gdk4-wayland-sys", version = "0.9" }
 x11-dl = "2.21"
 parking_lot = "0.12"
 dlopen2 = "0.7.0"

Cargo.toml

@@ -49,13 +49,13 @@ Gtk and its related libraries are used to build the support of Linux. Be sure to
 #### Arch Linux / Manjaro:
 
 ```bash
-sudo pacman -S gtk3
+sudo pacman -S gtk4
 ```
 
 #### Debian / Ubuntu:
 
 ```bash
-sudo apt install libgtk-3-dev
+sudo apt install libgtk-4-dev
 ```
 
 ### Acknowledgement

README.md

@@ -21,7 +21,9 @@ use crate::{
   error::{ExternalError, OsError},
   event_loop::{EventLoopBuilder, EventLoopWindowTarget},
   monitor::MonitorHandle,
-  platform_impl::{x11::xdisplay::XError, Parent, Window as UnixWindow},
+  platform_impl::{
+    gtk_window::ApplicationWindow, x11::xdisplay::XError, Parent, Window as UnixWindow,
+  },
   window::{Window, WindowBuilder},
 };
 
@@ -67,11 +69,11 @@ pub trait WindowExtUnix {
   /// and know what they're doing.
   fn new_from_gtk_window<T: 'static>(
     event_loop_window_target: &EventLoopWindowTarget<T>,
-    window: gtk::ApplicationWindow,
+    window: ApplicationWindow,
   ) -> Result<Window, OsError>;
 
   /// Returns the `gtk::ApplicatonWindow` from gtk crate that is used by this window.
-  fn gtk_window(&self) -> &gtk::ApplicationWindow;
+  fn gtk_window(&self) -> &ApplicationWindow;
 
   /// Returns the vertical `gtk::Box` that is added by default as the sole child of this window.
   /// Returns `None` if the default vertical `gtk::Box` creation was disabled by [`WindowBuilderExtUnix::with_default_vbox`].
@@ -84,7 +86,7 @@ pub trait WindowExtUnix {
 }
 
 impl WindowExtUnix for Window {
-  fn gtk_window(&self) -> &gtk::ApplicationWindow {
+  fn gtk_window(&self) -> &ApplicationWindow {
     &self.window.window
   }
 
@@ -98,7 +100,7 @@ impl WindowExtUnix for Window {
 
   fn new_from_gtk_window<T: 'static>(
     event_loop_window_target: &EventLoopWindowTarget<T>,
-    window: gtk::ApplicationWindow,
+    window: ApplicationWindow,
   ) -> Result<Window, OsError> {
     let window = UnixWindow::new_from_gtk_window(&event_loop_window_target.p, window)?;
     Ok(Window { window: window })
@@ -114,7 +116,7 @@ pub trait WindowBuilderExtUnix {
   fn with_skip_taskbar(self, skip: bool) -> WindowBuilder;
   /// Set this window as a transient dialog for `parent`
   /// <https://gtk-rs.org/gtk3-rs/stable/latest/docs/gdk/struct.Window.html#method.set_transient_for>
-  fn with_transient_for(self, parent: &impl gtk::glib::IsA<gtk::Window>) -> WindowBuilder;
+  fn with_transient_for(self, parent: &impl gtk::prelude::IsA<gtk::Window>) -> WindowBuilder;
 
   /// Whether to enable or disable the internal draw for transparent window.
   ///
@@ -158,8 +160,8 @@ impl WindowBuilderExtUnix for WindowBuilder {
     self
   }
 
-  fn with_transient_for(mut self, parent: &impl gtk::glib::IsA<gtk::Window>) -> WindowBuilder {
-    use gtk::glib::Cast;
+  fn with_transient_for(mut self, parent: &impl gtk::prelude::IsA<gtk::Window>) -> WindowBuilder {
+    use gtk::prelude::Cast;
     self.platform_specific.parent = Parent::ChildOf(parent.clone().upcast());
     self
   }

src/platform/unix.rs

@@ -3,16 +3,15 @@ use std::{
   ptr,
 };
 
-use gtk::glib;
 use x11_dl::{xinput2, xlib};
 
 use crate::event::{DeviceEvent, ElementState, RawKeyEvent};
 
 use super::keycode_from_scancode;
 
 /// Spawn Device event thread. Only works on x11 since wayland doesn't have such global events.
-pub fn spawn(device_tx: glib::Sender<DeviceEvent>) {
-  std::thread::spawn(move || unsafe {
+pub fn spawn(device_tx: async_channel::Sender<DeviceEvent>) {
+  std::thread::spawn(async move || unsafe {
     let xlib = xlib::Xlib::open().unwrap();
     let xinput2 = xinput2::XInput2::open().unwrap();
     let display = (xlib.XOpenDisplay)(ptr::null());
@@ -64,7 +63,7 @@ pub fn spawn(device_tx: glib::Sender<DeviceEvent>) {
                   state,
                 };
 
-                if let Err(e) = device_tx.send(DeviceEvent::Key(event)) {
+                if let Err(e) = device_tx.send_blocking(DeviceEvent::Key(event)) {
                   log::info!("Failed to send device event {} since receiver is closed. Closing x11 thread along with it", e);
                   break;
                 }