There was a mistake in my fix for wayland popups, as @b4n correctly noticed in https://github.com/geany/geany/pull/3011#issuecomment-1279825987. The problem was that the "Go to symbol definition" did not open the popup in correct location.
This PR contains two commits: - First fixes the problem - Second removes support for prehistoric versions of GTK
The first commit might be used separately, but I have to openly admit that the #ifdefed code is not tested, because I couldn't find any system in my reach with old enough GTK. The second commit should not be merged until GTK 3.24 is officially required for Geany. But I guess that it should not be a problem, this PR can probably wait untill then, since no one noticed the bug for a month. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/3316
-- Commit Summary --
* Fix go to symbol definition popup location * Drop support for GTK 3.21 and older
-- File Changes --
M src/callbacks.c (2) M src/editor.c (2) M src/msgwindow.c (9) M src/notebook.c (8) M src/plugins.c (3) M src/prefs.c (21) M src/sidebar.c (8) M src/symbols.c (91) M src/ui_utils.c (26) M src/ui_utils.h (2) M src/vte.c (2)
-- Patch Links --
https://github.com/geany/geany/pull/3316.patch https://github.com/geany/geany/pull/3316.diff