[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);

This requires GTK+ 3.22 but I think the build system only checks for GTK+ >= 3.0, so if someone built it with any version of GTK+ between 3.0 and 3.22, it would cause a build error.

-- 
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#pullrequestreview-259199360
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20190708/04acaa1f/attachment.html>