From 7855e7e042d9d99abb85f632198cb5ab868ec965 Mon Sep 17 00:00:00 2001
From: Evan Paterakis <evan@geopjr.dev>
Date: Tue, 1 Oct 2024 03:41:29 +0300
Subject: [PATCH] feat: FileChooserNative => FileDialog (#210)

---
 data/ui/application.ui  |  9 ------
 shard.lock              |  4 +--
 src/collision/window.cr | 62 ++++++++++++++++++-----------------------
 3 files changed, 29 insertions(+), 46 deletions(-)

diff --git a/data/ui/application.ui b/data/ui/application.ui
index 2c06fce..8889509 100644
--- a/data/ui/application.ui
+++ b/data/ui/application.ui
@@ -19,15 +19,6 @@
 			<attribute name="action">app.about</attribute>
 		</item>
 	</menu>
-	<object class="GtkFileChooserNative" id="mainFileChooserNative">
-		<property name="title" translatable="yes">Choose a File</property>
-		<property name="modal">1</property>
-		<property name="select-multiple">1</property>
-	</object>
-	<object class="GtkFileChooserNative" id="compareBtnFileChooserNative">
-		<property name="title" translatable="yes">Choose a File</property>
-		<property name="modal">1</property>
-	</object>
 	<template class="Collision-Window" parent="AdwApplicationWindow">
 		<property name="width-request">360</property>
 		<property name="height-request">360</property>
diff --git a/shard.lock b/shard.lock
index c924287..b3b105f 100644
--- a/shard.lock
+++ b/shard.lock
@@ -10,11 +10,11 @@ shards:
 
   gi-crystal:
     git: https://github.com/hugopl/gi-crystal.git
-    version: 0.22.3
+    version: 0.24.0
 
   gtk4:
     git: https://github.com/hugopl/gtk4.cr.git
-    version: 0.16.1
+    version: 0.17.0
 
   harfbuzz:
     git: https://github.com/hugopl/harfbuzz.cr.git
diff --git a/src/collision/window.cr b/src/collision/window.cr
index c01b593..8b15a00 100644
--- a/src/collision/window.cr
+++ b/src/collision/window.cr
@@ -6,8 +6,6 @@ module Collision
     resource: "/dev/geopjr/Collision/ui/application.ui",
     children: {
       "welcomeBtn",
-      "mainFileChooserNative",
-      "compareBtnFileChooserNative",
       "mainStack",
       "fileInfo",
       "headerbarStack",
@@ -35,8 +33,6 @@ module Collision
     @compareBtnLabel : Gtk::Label
     @compareBtnStack : Gtk::Stack
     @openFileBtn : Gtk::Button
-    @mainFileChooserNative : Gtk::FileChooserNative
-    @compareBtnFileChooserNative : Gtk::FileChooserNative
     @mainStack : Gtk::Stack
     @fileInfo : Adw::StatusPage
     @switcher_bar : Adw::ViewSwitcherBar
@@ -82,7 +78,23 @@ module Collision
     end
 
     def on_open_btn_clicked
-      @mainFileChooserNative.show
+      Gtk::FileDialog.new(
+        title: Gettext.gettext("Choose a File"),
+        modal: true
+      ).open_multiple(self, nil) do |obj, result|
+        next if (files = obj.as(Gtk::FileDialog).open_multiple_finish(result)).nil?
+
+        gio_files = [] of Gio::File
+        files.n_items.times do |i|
+          gio_files << Gio::File.cast(files.item(i).not_nil!)
+        end
+
+        loading
+        self.file = gio_files.shift
+        open_files(Adw::Application.cast(self.application.not_nil!), gio_files) unless gio_files.empty? || self.application.nil?
+      rescue ex
+        Collision::LOGGER.debug { ex }
+      end
     end
 
     def on_drop(file : Gio::File)
@@ -227,41 +239,12 @@ module Collision
       @compareBtnStack = Gtk::Stack.cast(template_child("compareBtnStack"))
       @progressbar = Gtk::ProgressBar.cast(template_child("progressbar"))
 
-      @mainFileChooserNative = Gtk::FileChooserNative.cast(template_child("mainFileChooserNative"))
-      @compareBtnFileChooserNative = Gtk::FileChooserNative.cast(template_child("compareBtnFileChooserNative"))
-      @mainFileChooserNative.transient_for = self
-      @compareBtnFileChooserNative.transient_for = self
-
       @verifyTextView.buffer.notify_signal["text"].connect do
         Collision::LOGGER.debug { "Verify tool notify event" }
 
         handle_input_change(@verifyTextView.buffer.text)
       end
 
-      @mainFileChooserNative.response_signal.connect do |response|
-        next unless response == -3
-
-        gio_files = [] of Gio::File
-        files = @mainFileChooserNative.files
-        files.n_items.times do |i|
-          gio_files << Gio::File.cast(files.item(i).not_nil!)
-        end
-
-        loading
-        self.file = gio_files.shift
-        open_files(Adw::Application.cast(self.application.not_nil!), gio_files) unless gio_files.empty? || self.application.nil?
-      rescue ex
-        Collision::LOGGER.debug { ex }
-      end
-
-      @compareBtnFileChooserNative.response_signal.connect do |response|
-        next unless response == -3
-
-        self.comparefile = @compareBtnFileChooserNative.file.not_nil!
-      rescue ex
-        Collision::LOGGER.debug { ex }
-      end
-
       @welcomeBtn.clicked_signal.connect(->on_open_btn_clicked)
       @openFileBtn.clicked_signal.connect(->on_open_btn_clicked)
 
@@ -269,7 +252,16 @@ module Collision
       @compareBtn.add_controller(Collision::DragNDrop.new(->comparefile=(Gio::File)).controller)
 
       @compareBtn.clicked_signal.connect do
-        @compareBtnFileChooserNative.show
+        Gtk::FileDialog.new(
+          title: Gettext.gettext("Choose a File"),
+          modal: true
+        ).open(self, nil) do |obj, result|
+          next if (file = obj.as(Gtk::FileDialog).open_finish(result)).nil?
+
+          self.comparefile = file
+        rescue ex
+          Collision::LOGGER.debug { ex }
+        end
       end
     end
   end