[geary] Remove conditional compilation for GTK+ 3.12.
- From: Michael Gratton <mjog src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [geary] Remove conditional compilation for GTK+ 3.12.
- Date: Sat, 21 May 2016 02:50:04 +0000 (UTC)
commit b7cf0ec663b9c68dee67469bea5056916cfc8b03
Author: Michael James Gratton <mike vee net>
Date: Sun Apr 10 00:44:15 2016 +1000
Remove conditional compilation for GTK+ 3.12.
CMakeLists.txt | 2 -
src/CMakeLists.txt | 9 -------
src/client/components/main-toolbar.vala | 31 +--------------------------
src/client/components/pill-toolbar.vala | 4 ---
src/client/composer/composer-headerbar.vala | 15 +------------
5 files changed, 2 insertions(+), 59 deletions(-)
---
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 0a59aee..b08a603 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -65,8 +65,6 @@ pkg_check_modules(WEBKITGTK22X QUIET webkitgtk-3.0>=2.2.0 webkitgtk-3.0<=2.2.100
pkg_check_modules(WEBKITGTK20X QUIET webkitgtk-3.0>=2.0.0 webkitgtk-3.0<=2.0.100)
pkg_check_modules(WEBKITGTK110X QUIET webkitgtk-3.0>=1.10.0 webkitgtk-3.0<=1.10.2)
-pkg_check_modules(GTK312X QUIET gtk+-3.0>=3.12.0)
-
pkg_check_modules(SQLITE311 QUIET sqlite3>=3.11.0)
pkg_check_modules(SQLITE312 QUIET sqlite3>=3.12.0)
if (SQLITE311_FOUND AND NOT SQLITE312_FOUND)
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index 9d89d3f..7e82f4b 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -518,15 +518,6 @@ else ()
message(FATAL_ERROR "WebKitGTK not installed / unknown version detected")
endif ()
-if (GTK312X_FOUND)
- message(STATUS "GTK+ 3.12 support: ON")
-
- set(EXTRA_VALA_OPTIONS
- ${EXTRA_VALA_OPTIONS}
- -D GTK_3_12
- )
-endif ()
-
if (NO_FATAL_WARNINGS)
message(STATUS "Vala fatal warnings: OFF")
else ()
diff --git a/src/client/components/main-toolbar.vala b/src/client/components/main-toolbar.vala
index 4fa9055..af6415e 100644
--- a/src/client/components/main-toolbar.vala
+++ b/src/client/components/main-toolbar.vala
@@ -87,15 +87,9 @@ public class MainToolbar : Gtk.Box {
insert.add(search_button);
Gtk.Box search = folder_header.create_pill_buttons(insert, false);
-#if !GTK_3_12
- folder_header.add_end(empty);
- folder_header.add_end(search);
- folder_header.add_end(new Gtk.Separator(Gtk.Orientation.VERTICAL));
-#else
folder_header.add_end(new Gtk.Separator(Gtk.Orientation.VERTICAL));
folder_header.add_end(search);
folder_header.add_end(empty);
-#endif
// Reply buttons
insert.clear();
@@ -127,25 +121,14 @@ public class MainToolbar : Gtk.Box {
false));
Gtk.Box undo = conversation_header.create_pill_buttons(insert);
- // pack_end() ordering is reversed in GtkHeaderBar in 3.12 and above
-#if !GTK_3_12
- conversation_header.add_end(archive_trash_delete);
- conversation_header.add_end(undo);
-#else
conversation_header.add_end(undo);
conversation_header.add_end(archive_trash_delete);
-#endif
pack_start(folder_header, false, false);
pack_start(conversation_header, true, true);
-#if GTK_3_12
Gtk.Settings.get_default().notify["gtk-decoration-layout"].connect(set_window_buttons);
realize.connect(set_window_buttons);
-#else
- get_style_context().changed.connect(set_close_buttons_side);
- realize.connect(set_close_buttons_side);
-#endif
}
/// Updates the trash button as trash or delete, and shows or hides the archive button.
@@ -164,9 +147,7 @@ public class MainToolbar : Gtk.Box {
guest_header_binding = bind_property("show-close-button-right", header,
"show-close-button", BindingFlags.SYNC_CREATE);
pack_start(header, true, true);
-#if GTK_3_12
header.decoration_layout = conversation_header.decoration_layout;
-#endif
}
public void remove_conversation_header(Gtk.HeaderBar header) {
@@ -175,13 +156,10 @@ public class MainToolbar : Gtk.Box {
header.get_style_context().remove_class("geary-titlebar-right");
GtkUtil.unbind(guest_header_binding);
header.show_close_button = false;
-#if GTK_3_12
header.decoration_layout = Gtk.Settings.get_default().gtk_decoration_layout;
-#endif
conversation_header.show();
}
-#if GTK_3_12
private void set_window_buttons() {
string[] buttons = Gtk.Settings.get_default().gtk_decoration_layout.split(":");
if (buttons.length != 2) {
@@ -193,12 +171,5 @@ public class MainToolbar : Gtk.Box {
folder_header.decoration_layout = buttons[0] + ":";
conversation_header.decoration_layout = ":" + buttons[1];
}
-#else
- private void set_close_buttons_side() {
- bool at_end = folder_header.close_button_at_end();
- show_close_button_left = show_close_button && !at_end;
- show_close_button_right = show_close_button && at_end;
- }
-#endif
-}
+}
diff --git a/src/client/components/pill-toolbar.vala b/src/client/components/pill-toolbar.vala
index 688937a..34c2988 100644
--- a/src/client/components/pill-toolbar.vala
+++ b/src/client/components/pill-toolbar.vala
@@ -133,11 +133,7 @@ public class PillHeaderbar : Gtk.HeaderBar, PillBar {
public bool close_button_at_end() {
string layout;
bool at_end = false;
-#if GTK_3_12
layout = Gtk.Settings.get_default().gtk_decoration_layout;
-#else
- get_toplevel().style_get("decoration-button-layout", out layout);
-#endif
// Based on logic of close_button_at_end in gtkheaderbar.c: Close button appears
// at end iff "close" follows a colon in the layout string.
if (layout != null) {
diff --git a/src/client/composer/composer-headerbar.vala b/src/client/composer/composer-headerbar.vala
index 9670ce2..947597b 100644
--- a/src/client/composer/composer-headerbar.vala
+++ b/src/client/composer/composer-headerbar.vala
@@ -83,29 +83,16 @@ public class ComposerHeaderbar : PillHeaderbar {
add_start(detach_start);
add_start(attach_buttons);
add_start(recipients);
-#if !GTK_3_12
- add_end(send_button);
- add_end(close_buttons);
- add_end(detach_end);
-#else
+
add_end(detach_end);
add_end(close_buttons);
add_end(send_button);
-#endif
-#if GTK_3_12
notify["decoration-layout"].connect(set_detach_button_side);
-#else
- get_style_context().changed.connect(set_detach_button_side);
-#endif
realize.connect(set_detach_button_side);
notify["state"].connect((s, p) => {
if (state == ComposerWidget.ComposerState.DETACHED) {
-#if GTK_3_12
notify["decoration-layout"].disconnect(set_detach_button_side);
-#else
- get_style_context().changed.disconnect(set_detach_button_side);
-#endif
detach_start.visible = detach_end.visible = false;
}
});
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]