Created attachment 8668 Patch for described changes Some use cases of Directory Menu may not require Open Folder/Open in Terminal commands. For example I use Directory Menu as a way to list txt files with notes. In such case removal of menu items makes for better experience. I prepared patch, which enables user to switch off displaying those menu items through plugin properties.
Thanks for the patch! While it mostly works as expected, the separator that is above the "Open Folder" / "Open in Terminal" menuitems always remains visible, which looks strange. Would you mind updating your patch to take this into account?
> Would you mind updating your patch to take this into account? Thank you for taking care :) Actually I thought that leaving separator allows users to better recognize that the menu is absent and the 1st item is file/directory item and not menu item. But no problem, I will add code to hide it as you suggest.
Just a reminder to amend your patch so it can get merged.
Created attachment 9297 Patch updated with separator hiding
Simon Steinbeiss referenced this bugreport in commit a13494d184b8888f16d080dbd03ab79fb33a0f46 directorymenu: Add option to hide folder/terminal menuitems (Bug #15630) https://git.xfce.org/xfce/xfce4-panel/commit?id=a13494d184b8888f16d080dbd03ab79fb33a0f46
I apologize for the slightly hasty push - I forgot to change the patch author to your name :/ (I had to fix the wording of the menuitems because you spelled "Folder" vs "folder" inconsistently in your patch.) I hope you still appreciate the merging of your patch! I'm on to review your second patch now...
(In reply to Simon Steinbeiss from comment #6) > I apologize for the slightly hasty push - I forgot to change the patch > author to your name :/ Not a problem, really. Don't waste your time changing it. > I hope you still appreciate the merging of your patch! > I'm on to review your second patch now... Yes, that's perfect! Thank you!