[Github-comments] Re: [geany/geany] preference option for file display size (PR #3365)

2023-01-15 Thread elextr via Github-comments
@elextr commented on this pull request. > @@ -434,7 +434,7 @@ void document_update_tab_label(GeanyDocument *doc) g_return_if_fail(doc != NULL); - short_name = document_get_basename_for_display(doc, -1); + short_name = document_get_basename_for_display(doc,

[Github-comments] Re: [geany/geany-plugins] Remove unusable plugins: DevHelp, GeanyPy, Multiterm (PR #1217)

2023-01-15 Thread elextr via Github-comments
Well to be fair, Geanypy has a dependency that has stated it will never be newer than GTK2, so "upgrading" would be a rewrite. So since it was a rewrite @kugel- replaced Geanypy with Peasy, which is also better, just its not been added to GP. I think its in some distros (Arch AUR at least),

[Github-comments] Re: [geany/geany-plugins] GeanyLua: Use luaL_register instead of luaL_openlib (PR #1210)

2023-01-15 Thread Enrico Tröger via Github-comments
Thank you! -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1210#issuecomment-1383212166 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] GeanyLua: Use luaL_register instead of luaL_openlib (PR #1210)

2023-01-15 Thread Enrico Tröger via Github-comments
Merged #1210 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1210#event-8230091512 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Use constructor method calltip only if available (PR #3334)

2023-01-15 Thread Enrico Tröger via Github-comments
Merged #3334 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3334#event-8230062418 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] GeanyLua: Use luaL_register instead of luaL_openlib (PR #1210)

2023-01-15 Thread Chris Mayo via Github-comments
Oops, sorry you're right. There was a luaL_openlib***s***, converted incorrectly. PR updated to put that back: ```diff --- a/geanylua/glspi_run.c +++ b/geanylua/glspi_run.c @@ -246,7 +246,7 @@ static lua_State *glspi_state_new(void) { lua_State *L = luaL_newstate();

[Github-comments] Re: [geany/geany] CI: Use MSYS2 for Windows build (PR #3315)

2023-01-15 Thread Enrico Tröger via Github-comments
Merged #3315 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3315#event-8230023077 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] CI: Use MSYS2 for Windows build (PR #3315)

2023-01-15 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. e2066d4f5f9600b0d8acd0ebeaeddebc93d1f15a Use Ubuntu 20.04 for Linux CI builds -- View it on GitHub: https://github.com/geany/geany/pull/3315/files/f146c2eb70715e94d1e31931cdebb08c86f81799..e2066d4f5f9600b0d8acd0ebeaeddebc93d1f15a You are receiving this because you are

[Github-comments] Re: [geany/infrastructure] Builders: Use the "pacman" package from Debian to avoid building it (PR #8)

2023-01-15 Thread Enrico Tröger via Github-comments
Merged #8 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/infrastructure/pull/8#event-8229970482 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] GeanyLua: Use luaL_register instead of luaL_openlib (PR #1210)

2023-01-15 Thread Enrico Tröger via Github-comments
Did the changes work for you? With these changes, Geany crashes when using any of the example scripts. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1210#issuecomment-1383200490 You are receiving this because you are subscribed to this thread.

[Github-comments] Re: [geany/geany-plugins] Remove unusable plugins: DevHelp, GeanyPy, Multiterm (PR #1217)

2023-01-15 Thread Enrico Tröger via Github-comments
It feels weird to delete plugins from @b4n, @kugel- and @codebrainz :upside_down_face: I guess you do not intend to update any of those plugins? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1217#issuecomment-1383195689 You are receiving

[Github-comments] Re: [geany/geany] How reduce scrolling speed in Geany ? (#2768)

2023-01-15 Thread Enrico Tröger via Github-comments
I did my best to create https://sourceforge.net/p/scintilla/bugs/2375/, @elextr and @nyamatongwe. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2768#issuecomment-1383179337 You are receiving this because you are subscribed to this thread. Message

[Github-comments] Re: [geany/geany] How reduce scrolling speed in Geany ? (#2768)

2023-01-15 Thread Enrico Tröger via Github-comments
Reopened #2768. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2768#event-8229775183 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] preference option for file display size (PR #3365)

2023-01-15 Thread Enrico Tröger via Github-comments
@eht16 requested changes on this pull request. Apart from the few comments, it looks very good. By accident I found https://github.com/geany/geany/issues/2112 which would be solved by this as well. Maybe you could add "closes #2112 and #3344" to the commit message and rephrase it a bit as