Re: [Github-comments] [geany/geany] Allow “freezing” project session (#1397)
You could do this fairly trivially in a plugin. -- 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/issues/1397#issuecomment-280254457
Re: [Github-comments] [geany/geany] Allow “freezing” project session (#1397)
Geany projects are basically named sessions, so it seems kind of an anti-use-case to not save the session, it is likely to only be accepted if it has an option and the default is current behaviour. -- 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/issues/1397#issuecomment-280253381
Re: [Github-comments] [geany/geany] Allow “freezing” project session (#1397)
I’m sorry, I wasn’t clear. It’s not the files themselves that I want to make read-only — it’s the *set* of open files, i.e. the project session. So if I open some other files while working on the project, Geany shouldn’t remember that. -- 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/issues/1397#issuecomment-280252203
Re: [Github-comments] [geany/geany] Feature request: use split window for "Go to Symbol Definition" (#1394)
I would use this feature but I'm not fond of putting a preference in the main UI like that. It would probably fit better in a plugin preferences tab. More importantly though I think it would be hard to implement cleanly/easily as the core of Geany would have to try and let plugins handle going to definition and if none did then it would have to fall back to the original behaviour. This sort of sound like the "providers" API that was previously discussed for plugins [on the mailing list](https://www.mail-archive.com/devel@lists.geany.org/msg02841.html) (TLDR; allowing plugins to hook in to builtin features like this). -- 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/issues/1394#issuecomment-280217547
Re: [Github-comments] [geany/geany] Sync ctags main part (#1263)
> By the way, has anything happened with travis? Seems no checks are running. @eht16 mode some changes to the organisation IIRC, did that break it? -- 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/pull/1263#issuecomment-280192213
Re: [Github-comments] [geany/geany] Fix setting menu item labels dynamically (#1396)
elextr requested changes on this pull request. Needs to set use_underline so the mnemonics work, thats the reason for the workaround in the first place. -- 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/pull/1396#pullrequestreview-22144225
Re: [Github-comments] [geany/geany] Allow “freezing” project session (#1397)
What wrong with setting the files you have open for "tags only" purposes readonly (Menu->Document->Read Only), then you can't change them and they won't get saved. -- 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/issues/1397#issuecomment-280182267
Re: [Github-comments] [geany/geany] Add more info to the goto symbol popup (#1398)
The idea sounds useful, for languages that don't have to put every class in its own file. My current project has `parse()` in each of ten classes. Maybe replace the filename with the scope otherwise where the scope is a nested thing (eg `my_namespace::my_sub_namespace::my_class::my_nested_class`) the popup may get a bit big. Or have preferences that allow the user to choose one or both and can then the list be expanded to add any other potential disambiguating information. -- 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/issues/1398#issuecomment-280180672
[Github-comments] [geany/geany] Add more info to the goto symbol popup (#1398)
To expand on #950 and #1069... As a Python programmer, I find that the goto symbol popup is considerably more useful to me with a change like this: ``` - text = g_markup_printf_escaped(_("%s: %lu"), fname, tmtag->line); + text = g_markup_printf_escaped( + _("%s\n%s: %lu"), + tmtag->scope, fname, tmtag->line); ``` because then I can immediately see the name of the class containing the tag, which is useful when I have 10 different classes with an `execute` method in each of them. But this doesn’t work in many other cases (even in Python). Then of course, `TMTag` has a bunch of other fields that might also be very useful. Maybe there’s a way to put this information to good use without breaking the current use cases? Maybe a preference? -- 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/issues/1398
[Github-comments] [geany/geany] Allow “freezing” project session (#1397)
Geany has a preference called “Use project-based session files”. When it’s enabled, a project’s open files are saved on project close and reopened on project open. I would like a per-project setting to change this behavior such that open files are **opened, but not saved**. In other words, I want to “freeze” a specific set of files to be opened with the project. My use case: a blunt way to preload a project’s local tags. I have several projects consisting of a few dozen source files each, and I find that simply opening them all at startup is a good way to have all the tags handy. I don’t want heavyweight approaches like Project Organizer. But I can think of other use cases as well. Right now I have a blunt patch that relies on the user manually writing `readonly_session=1` in the `.geany` file. I guess that wouldn’t be acceptable for Geany core? What do you think the UI for this feature should look like? Or is it appropriate for Geany core at all? -- 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/issues/1397
[Github-comments] [geany/geany] Fix setting menu item labels dynamically (#1396)
Fixes https://github.com/geany/geany-plugins/issues/462 You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/1396 -- Commit Summary -- * Fix setting menu item labels dynamically -- File Changes -- M src/build.c (15) -- Patch Links -- https://github.com/geany/geany/pull/1396.patch https://github.com/geany/geany/pull/1396.diff -- 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/pull/1396
Re: [Github-comments] [geany/geany-plugins] TreeBrowser: fix refreshing without selection (#525)
Merged #525. -- 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/525#event-963945600
[Github-comments] [geany/geany-plugins] TreeBrowser: fix refreshing without selection (#525)
This fixes a bug that could be reproduced as follows: 1. ensure that *no* item in the tree is selected (highlighted); 2. press the "Refresh" keybinding. Without this fix, the contents of the tree are duplicated. You can view, comment on, or merge this pull request online at: https://github.com/geany/geany-plugins/pull/525 -- Commit Summary -- * TreeBrowser: fix refreshing without selection -- File Changes -- M treebrowser/src/treebrowser.c (3) -- Patch Links -- https://github.com/geany/geany-plugins/pull/525.patch https://github.com/geany/geany-plugins/pull/525.diff -- 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/525
[Github-comments] [geany/geany-plugins] TreeBrowser: keybinding to track current file (#524)
TreeBrowser has a "Follow current document" option, but I don't use it because it causes the tree's root to change. Still, sometimes I have a large tree and I want to quickly find the current document in it. You can view, comment on, or merge this pull request online at: https://github.com/geany/geany-plugins/pull/524 -- Commit Summary -- * TreeBrowser: keybinding to track current file -- File Changes -- M treebrowser/src/treebrowser.c (7) -- Patch Links -- https://github.com/geany/geany-plugins/pull/524.patch https://github.com/geany/geany-plugins/pull/524.diff -- 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/524
Re: [Github-comments] [geany/geany-plugins] Use online version of plugin help (#521)
Merged #521. -- 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/521#event-963876843
Re: [Github-comments] [geany/geany-plugins] TreeBrowser: pop up context menu on Shift+F10 (#523)
Merged #523. -- 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/523#event-963875239
Re: [Github-comments] [geany/geany] Geany 1.28 takes a lot of time to open on macOS Sierra (#1277)
I asked on gtk-osx list: https://mail.gnome.org/archives/gtk-osx-users-list/2017-February/msg0.html -- 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/issues/1277#issuecomment-280146862
Re: [Github-comments] [geany/geany-plugins] TreeBrowser: more mnemonics in the context menu (#522)
Merged #522. -- 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/522#event-963863996
[Github-comments] [geany/geany-plugins] TreeBrowser: pop up context menu on Shift+F10 (#523)
GTK+ by default maps Shift+F10 to the same thing as Menu, which is useful for laptop keyboards (that have no Menu key). But because TreeBrowser handles keypresses on its own, it doesn't get this functionality automatically. Implementation is per the "Test for modifier keys correctly" section of the GTK+ reference manual. You can view, comment on, or merge this pull request online at: https://github.com/geany/geany-plugins/pull/523 -- Commit Summary -- * TreeBrowser: pop up context menu on Shift+F10 -- File Changes -- M treebrowser/src/treebrowser.c (4) -- Patch Links -- https://github.com/geany/geany-plugins/pull/523.patch https://github.com/geany/geany-plugins/pull/523.diff -- 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/523
[Github-comments] [geany/geany-plugins] TreeBrowser: more mnemonics in the context menu (#522)
This menu has many useful items (I use at least 6 of them daily), but finding them manually with Up/Down or the mouse is annoying. The choice of mnemonics is somewhat arbitrary, but they are unique. Note that TreeBrowser has keybindings for a few of these operations, but they are not quite the same thing, because keybindings are global to Geany, and the global keybinding space is sort of crowded for me already (hard to pick a convenient but unused one). You can view, comment on, or merge this pull request online at: https://github.com/geany/geany-plugins/pull/522 -- Commit Summary -- * TreeBrowser: more mnemonics in the context menu -- File Changes -- M treebrowser/src/treebrowser.c (30) -- Patch Links -- https://github.com/geany/geany-plugins/pull/522.patch https://github.com/geany/geany-plugins/pull/522.diff -- 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/522
Re: [Github-comments] [geany/geany] Geany 1.28 takes a lot of time to open on macOS Sierra (#1277)
OK, some more info: it's gdk_display_get_name() here https://github.com/geany/geany/blob/master/src/socket.c#L288 which is slow. After commenting-out and leaving NODISPLAY, it starts fast. I'll try to put traces into GDK if I find something. In the worst case the gdk_display_get_name() call could just be ifdefed on OS X. -- 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/issues/1277#issuecomment-280133984
Re: [Github-comments] [geany/geany] Geany 1.28 takes a lot of time to open on macOS Sierra (#1277)
Alright, just tried to put some traces into Geany and it seems to have something to do with the socket that Geany uses to open an existing instance if it's already running - when I remove the code, it starts faster. It's also something I noticed on the latest macOS release that Geany always opens a new instance instead of existing instance so there seems to be something broken with the socket. I'll investigate more. -- 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/issues/1277#issuecomment-280128900
Re: [Github-comments] [geany/geany] Geany 1.28 takes a lot of time to open on macOS Sierra (#1277)
Ok, then I'm out of ideas as well :(. -- 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/issues/1277#issuecomment-280124724
Re: [Github-comments] [geany/geany] Geany 1.28 takes a lot of time to open on macOS Sierra (#1277)
No, this also happens without any plugins activated. Also the same binary works fine on previous versions of OS X without any delay (have older versions of OS X in virtual machines). -- 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/issues/1277#issuecomment-280124331
Re: [Github-comments] [geany/geany] Geany 1.28 takes a lot of time to open on macOS Sierra (#1277)
Are plugins enabled? Maybe some Webkit-enabled plugin tries to do some network I/O and fail or timeout in some way? On Windows, there is a noticeable delay when loading any Webkit-enabled plugin whereas there it seems to be just loading the libraries which needs CPU time (and so probably not the same issue as here). -- 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/issues/1277#issuecomment-280123528
Re: [Github-comments] [geany/geany] Keyboard shortcuts for menu items (#1324)
@thewhitetulip We all hate you! ;-) -- 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/issues/1324#issuecomment-280091501
Re: [Github-comments] [geany/geany-plugins] Project Organizer: Help-Button not working (#507)
I went through other plugins and most of them use the online html version of the documentation which is probably the easiest solution. I created a pull request for geanyctags and projectorganizer here: https://github.com/geany/geany-plugins/pull/521. -- 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/issues/507#issuecomment-280090737
Re: [Github-comments] [geany/geany] Keyboard shortcuts for menu items (#1324)
@techee I am sorry, I switched to vim. -- 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/issues/1324#issuecomment-280064821
Re: [Github-comments] [geany/geany] Sync ctags main part (#1263)
Great! Yeah, pure fear of the size of the patch made me not to touch it for such a long time (and it wasn't that bad at the end, thought it would take me a week to remember what it does). By the way, has anything happened with travis? Seems no checks are running. -- 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/pull/1263#issuecomment-280055679
Re: [Github-comments] [geany/geany] Geany 1.28 takes a lot of time to open on macOS Sierra (#1277)
Sorry, I didn't notice you opened this bug until now. I've seen something similar and frankly have no idea what the reason could be. It doesn't seem to be CPU-bound as CPU doesn't seem to be used much during start. Neither anything I/O-intensive seems to happen. I also thought it might be related to signature verification but I get the same times also without signing the binary. Assigning to myself though no idea what to do about it. -- 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/issues/1277#issuecomment-280048277
Re: [Github-comments] [geany/geany] Keyboard shortcuts for menu items (#1324)
I created a pull request here https://github.com/geany/geany/pull/1395 which adds basic support to enable platform-specific keybinding overrides. I didn't create the configuration with overrides though, for what I use I'm fine with the default ones. @thewhitetulip if you are still interested in this feature, please create a patch which updates the keybindings_osx.conf file. -- 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/issues/1324#issuecomment-280043830
[Github-comments] [geany/geany] Add support for platform-specific keybindings (#1395)
Allow platform-specific keybinding overrides. At the moment only for OS X where the keybindings can be set inside keybindings_osx.conf. Similar config files can be added for other platforms if needed in the future. The only change is the new init_platform_kb() function which is called inside keybindings_init() (had to move it down a bit in the source code to avoid forward declarations but there's no other change in this function). You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/1395 -- Commit Summary -- * Add support for platform-specific keybindings -- File Changes -- M data/Makefile.am (1) A data/keybindings_osx.conf (1) M src/keybindings.c (44) -- Patch Links -- https://github.com/geany/geany/pull/1395.patch https://github.com/geany/geany/pull/1395.diff -- 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/pull/1395
Re: [Github-comments] [geany/geany] Sync ctags main part (#1263)
@techee awesome :) It'll take me a little time to also get back to that well enough to understand my questions and your answers, but I'll try and do that as soon as possible. -- 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/pull/1263#issuecomment-280013961
Re: [Github-comments] [geany/geany] Go to previous cursor position (#1359)
I'd suggest only saving the caret position when the its line changes. Also, I'd feel very comfortable if history navigation was bound to the (now common) buttons 4 and 5 of the mouse, like in any current internet browser. -- 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/issues/1359#issuecomment-279985524
Re: [Github-comments] [geany/geany] Sync ctags main part (#1263)
@techee pushed 9 commits. 8455f8e lcpp, c: Fix signature collection d5823ce c.c: Reinitialize rescan variable in case longjmp() happens 634719a c.c: Eliminate useless kind_for_define variable cf6653a Don't check if kind is enabled for library 6176846 Remove unnecessary use of AH_TEMPLATE 3e8365c asciidoc, rest: Remove unnecessary scope assignments e5b5326 python: Rename corkPair struct to corkInfo df0ae9b python: minor cleanups bf4eb8f lcpp: use eStrdup() instead of strdup() -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/geany/geany/pull/1263/files/0a6acff8186aa372e8a906ad627acf27fa9dbad8..bf4eb8f093ddcf8abec947397ab57a8bb8108800