@LarsGit223 requested changes on this pull request.

The gtk version checks are not correct. This will break on systems between gtk version 3.0 and 3.4.


In src/tools.c:

> @@ -828,20 +842,27 @@ void tools_color_chooser(const gchar *color)
 
 	if (ui_widgets.open_colorsel == NULL)
 	{
+#if (gtk_major_version == 3)	
+		ui_widgets.open_colorsel = gtk_color_chooser_dialog_new(_("Color Chooser"), GTK_WINDOW(main_widgets.window));

gtk_color_chooser_dialog_new() requires at least gtk version 3.4 not 3. You could use #if GTK_CHECK_VERSION(3, 4, 0) ... else ... endif for a correct version check.


In src/tools.c:

>  			GeanyDocument *doc = document_get_current();
+			g_return_if_fail(doc != NULL);
+#if (gtk_major_version == 3)
+			GdkRGBA color;
+			char hex[8] = { 0 };
+
+			gtk_color_chooser_get_rgba(GTK_COLOR_CHOOSER(ui_widgets.open_colorsel), &color);
+			sprintf(hex, "#%02X%02X%02X",

This requires version 3.4 not 3.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.