@elextr requested changes on this pull request.
In principle the right click menu should be context related, like the `Open
Selected File` entry[^1]. So it might be better to add these new items to a
normal menu, and then they can be given a keyboard accelerator and even a
shortcut as well.
I suggest that adding a separate command for terminal would be better than
using the existing command and substituting nothing for `%c`. There is no
guarantee that will produce a valid command. There is plenty of space in the
Tools Preferences page.
Which version of glade did you use? there seems to be a lot of unrelated change
noise.
I have only had a quick look review so far.
[^1]: Yes yes I know it has accumulated non-context related things over the
years, but thats no excuse to continue the trend.
> in the command are substituted with the run script name, see
`Terminal emulators`_.
+File Manager
+ The command to use to open a file manager. Occurences of %d
+ are substituted with a directory in which to open it.
+
Say _what_ is substituted for %d directory of current file or home dir if no
files open, its up to the command to determine what to do with that.
> @@ -73,37 +73,40 @@
/* some default settings which are used at the very first start of Geany to
fill
* the configuration file */
-#define GEANY_MAX_SYMBOLLIST_HEIGHT 10
-#define GEANY_MIN_SYMBOLLIST_CHARS 4
-#define GEANY_MSGWIN_HEIGHT 208
-#define GEANY_DISK_CHECK_TIMEOUT 30
-#define GEANY_DEFAULT_TOOLS_MAKE "make"
+#define GEANY_MAX_SYMBOLLIST_HEIGHT 10
+#define GEANY_MIN_SYMBOLLIST_CHARS 4
+#define GEANY_MSGWIN_HEIGHT 208
+#define GEANY_DISK_CHECK_TIMEOUT 30
+#define GEANY_DEFAULT_TOOLS_MAKE "make"
Why all these noise changes?
> gchar *term_cmd; /**<
> terminal emulator command */
+ gchar *file_manager_cmd; /**< file
manager command */
This struct is in the plugin API, so add new fields at the end to avoid
breaking the ABI, increment the API, see HACKING.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3813#pullrequestreview-1981492918
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/pull/3813/review/[email protected]>
