[Github-comments] [geany/geany-plugins] MultiTerm: Add support for GTK3 (#95)

[Matthew Brush]

codebrainz commented on this pull request.



> @@ -149,7 +153,11 @@ namespace MultiTerm
 				context_menu.new_window_activate.connect(on_new_window_activate);
 				context_menu.move_to_location_activate.connect(on_move_to_location);
 			}
+#if MULTITERM_GTK3
+			context_menu.popup_at_pointer(event);

Not for compile-time versions (ie. `GTK_CHECK_VERSION` macros), Vala's preprocessor is too wimpy. You can pass along a macro from the build system using `-D` option if it's really needed.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/95#discussion_r302785044
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20190711/58633b0a/attachment.html>