Adjust to Adw 1.4: Use specialized ActionRows

7 files changed, 16 insertions, 59 deletions

diff --git a/main/data/preferences_window/general_preferences_page.ui b/main/data/preferences_window/general_preferences_page.ui
index 33d1a2c9..9e76e17a 100644
--- a/main/data/preferences_window/general_preferences_page.ui
+++ b/main/data/preferences_window/general_preferences_page.ui
@@ -5,28 +5,15 @@
     <child>
       <object class="AdwPreferencesGroup">
         <child>
-          <object class="AdwActionRow">
+          <object class="AdwSwitchRow" id="typing_row">
             <property name="title" translatable="yes">Send _Typing Notifications</property>
             <property name="use-underline">True</property>
-            <property name="activatable-widget">typing_switch</property>
-            <child type="suffix">
-              <object class="GtkSwitch" id="typing_switch">
-<!--                <property name="active" bind-source="DinoUiGeneralPreferencesPage" bind-property="send-typing" bind-flags="sync-create|bidirectional" />-->
-                <property name="valign">center</property>
-              </object>
-            </child>
           </object>
         </child>
         <child>
-          <object class="AdwActionRow">
+          <object class="AdwSwitchRow" id="marker_row">
             <property name="title" translatable="yes">Send _Read Receipts</property>
             <property name="use-underline">True</property>
-            <property name="activatable-widget">marker_switch</property>
-            <child type="suffix">
-              <object class="GtkSwitch" id="marker_switch">
-                <property name="valign">center</property>
-              </object>
-            </child>
           </object>
         </child>
       </object>
@@ -34,16 +21,10 @@
     <child>
       <object class="AdwPreferencesGroup">
         <child>
-          <object class="AdwActionRow">
+          <object class="AdwSwitchRow" id="notification_row">
             <property name="title" translatable="yes">_Notifications</property>
             <property name="subtitle" translatable="yes">Notify when a new message arrives</property>
             <property name="use-underline">True</property>
-            <property name="activatable-widget">notification_switch</property>
-            <child type="suffix">
-              <object class="GtkSwitch" id="notification_switch">
-                <property name="valign">center</property>
-              </object>
-            </child>
           </object>
         </child>
       </object>
@@ -51,15 +32,9 @@
     <child>
       <object class="AdwPreferencesGroup">
         <child>
-          <object class="AdwActionRow">
+          <object class="AdwSwitchRow" id="emoji_row">
             <property name="title" translatable="yes">_Convert Smileys to Emoji</property>
             <property name="use-underline">True</property>
-            <property name="activatable-widget">emoji_switch</property>
-            <child type="suffix">
-              <object class="GtkSwitch" id="emoji_switch">
-                <property name="valign">center</property>
-              </object>
-            </child>
           </object>
         </child>
       </object>
diff --git a/main/meson.build b/main/meson.build
index be7e531c..af9523e5 100644
--- a/main/meson.build
+++ b/main/meson.build
@@ -127,9 +127,6 @@ if dep_gtk4.version() == 'unknown' or dep_gtk4.version().version_compare('>=4.14
     vala_args += ['-D', 'GTK_4_14']
     sources += files('src/ui/conversation_content_view/file_transmission_progress.c')
 endif
-if dep_libadwaita.version() == 'unknown' or dep_libadwaita.version().version_compare('>=1.3')
-    vala_args += ['-D', 'Adw_1_3']
-endif
 if dep_libadwaita.version() == 'unknown' or dep_libadwaita.version().version_compare('>=1.4')
     vala_args += ['-D', 'Adw_1_4']
 endif
diff --git a/main/src/ui/util/preference_group.vala b/main/src/ui/util/preference_group.vala
index 6056f2c8..efb5253a 100644
--- a/main/src/ui/util/preference_group.vala
+++ b/main/src/ui/util/preference_group.vala
@@ -61,9 +61,7 @@ namespace Dino.Ui.Util {
             var view = new Adw.ActionRow() {
                 title = row_text.title,
                 subtitle = row_text.text,
-#if Adw_1_3
-                subtitle_selectable = true,
-#endif
+                subtitle_selectable = true
             };
             view.add_css_class("property");
 
@@ -88,17 +86,9 @@ namespace Dino.Ui.Util {
             foreach (string text in combobox_view_model.items) {
                 string_list.append(text);
             }
-#if Adw_1_4
             var view = new Adw.ComboRow() { title = combobox_view_model.title };
             view.model = string_list;
             combobox_view_model.bind_property("active-item", view, "selected", BindingFlags.SYNC_CREATE | BindingFlags.BIDIRECTIONAL);
-#else
-            var view = new Adw.ActionRow() { title = combobox_view_model.title };
-            var drop_down = new DropDown(string_list, null) { valign = Align.CENTER };
-            combobox_view_model.bind_property("active-item", drop_down, "selected", BindingFlags.SYNC_CREATE | BindingFlags.BIDIRECTIONAL);
-            view.activatable_widget = drop_down;
-            view.add_suffix(drop_down);
-#endif
             return view;
         }
 
diff --git a/main/src/windows/conversation_details.vala b/main/src/windows/conversation_details.vala
index f6f420d2..bce9c423 100644
--- a/main/src/windows/conversation_details.vala
+++ b/main/src/windows/conversation_details.vala
@@ -78,10 +78,9 @@ namespace Dino.Ui.ConversationDetails {
             }
             block_button.menu_model = block_menu_model;
 
-#if Adw_1_4
             // TODO: replace with putting buttons in new line on small screens
             notification_button_menu_content.can_shrink = true;
-#endif
+
             update_blocked_button();
         }
 
diff --git a/main/src/windows/preferences_window/account_preferences_subpage.vala b/main/src/windows/preferences_window/account_preferences_subpage.vala
index 1a7d5696..92fffa76 100644
--- a/main/src/windows/preferences_window/account_preferences_subpage.vala
+++ b/main/src/windows/preferences_window/account_preferences_subpage.vala
@@ -31,13 +31,11 @@ public class Dino.Ui.AccountPreferencesSubpage : Gtk.Box {
     private ulong alias_entry_changed = 0;
 
     construct {
-#if Adw_1_4
         headerbar.show_title = false;
-#endif
         button_container.layout_manager = new NaturalDirectionBoxLayout((BoxLayout)button_container.layout_manager);
         back_button.clicked.connect(() => {
             var window = (Adw.PreferencesWindow) this.get_root();
-            window.close_subpage();
+            window.pop_subpage();
         });
         edit_avatar_button.clicked.connect(() => {
             show_select_avatar();
@@ -173,8 +171,7 @@ public class Dino.Ui.AccountPreferencesSubpage : Gtk.Box {
                 model.remove_account(account);
                 // Close the account subpage
                 var window = (Adw.PreferencesWindow) this.get_root();
-                window.close_subpage();
-//                window.pop_subpage();
+                window.pop_subpage();
             }
             dialog.close();
         });
diff --git a/main/src/windows/preferences_window/general_preferences_page.vala b/main/src/windows/preferences_window/general_preferences_page.vala
index 6f8dd771..1e81a1b7 100644
--- a/main/src/windows/preferences_window/general_preferences_page.vala
+++ b/main/src/windows/preferences_window/general_preferences_page.vala
@@ -9,10 +9,10 @@ public class Dino.Ui.ViewModel.GeneralPreferencesPage : Object {
 
 [GtkTemplate (ui = "/im/dino/Dino/preferences_window/general_preferences_page.ui")]
 public class Dino.Ui.GeneralPreferencesPage : Adw.PreferencesPage {
-    [GtkChild] private unowned Switch typing_switch;
-    [GtkChild] private unowned Switch marker_switch;
-    [GtkChild] private unowned Switch notification_switch;
-    [GtkChild] private unowned Switch emoji_switch;
+    [GtkChild] private unowned Adw.SwitchRow typing_row;
+    [GtkChild] private unowned Adw.SwitchRow marker_row;
+    [GtkChild] private unowned Adw.SwitchRow notification_row;
+    [GtkChild] private unowned Adw.SwitchRow emoji_row;
 
     public ViewModel.GeneralPreferencesPage model { get; set; default = new ViewModel.GeneralPreferencesPage(); }
     private Binding[] model_bindings = new Binding[0];
@@ -27,10 +27,10 @@ public class Dino.Ui.GeneralPreferencesPage : Adw.PreferencesPage {
         }
         if (model != null) {
             model_bindings = new Binding[] {
-                model.bind_property("send-typing", typing_switch, "active", BindingFlags.SYNC_CREATE | BindingFlags.BIDIRECTIONAL),
-                model.bind_property("send-marker", marker_switch, "active", BindingFlags.SYNC_CREATE | BindingFlags.BIDIRECTIONAL),
-                model.bind_property("notifications", notification_switch, "active", BindingFlags.SYNC_CREATE | BindingFlags.BIDIRECTIONAL),
-                model.bind_property("convert-emojis", emoji_switch, "active", BindingFlags.SYNC_CREATE | BindingFlags.BIDIRECTIONAL)
+                model.bind_property("send-typing", typing_row, "active", BindingFlags.SYNC_CREATE | BindingFlags.BIDIRECTIONAL),
+                model.bind_property("send-marker", marker_row, "active", BindingFlags.SYNC_CREATE | BindingFlags.BIDIRECTIONAL),
+                model.bind_property("notifications", notification_row, "active", BindingFlags.SYNC_CREATE | BindingFlags.BIDIRECTIONAL),
+                model.bind_property("convert-emojis", emoji_row, "active", BindingFlags.SYNC_CREATE | BindingFlags.BIDIRECTIONAL)
             };
         } else {
             model_bindings = new Binding[0];
diff --git a/main/src/windows/preferences_window/preferences_window.vala b/main/src/windows/preferences_window/preferences_window.vala
index 662a6924..cee4e34e 100644
--- a/main/src/windows/preferences_window/preferences_window.vala
+++ b/main/src/windows/preferences_window/preferences_window.vala
@@ -15,7 +15,6 @@ public class Dino.Ui.PreferencesWindow : Adw.PreferencesWindow {
     [GtkChild] public unowned ViewModel.PreferencesWindow model { get; }
 
     construct {
-        this.can_navigate_back = true; // remove once we require Adw > 1.4
         this.bind_property("model", accounts_page, "model", BindingFlags.SYNC_CREATE);
         this.bind_property("model", account_page, "model", BindingFlags.SYNC_CREATE);
         this.bind_property("model", encryption_page, "model", BindingFlags.SYNC_CREATE);