search

hi5 / F4MiniMenu

Open selected file(s) from TC in defined editor(s) - A minimalistic clone of F4Menu 💾
http://ghisler.ch/board/viewtopic.php?t=35721
MIT License
59 stars 4 forks source link

Add basic support for Explorer, Double Commander, and XYPlorer #24

Open hi5 opened 3 years ago

hi5 commented 3 years ago

In the next version you will be able to use the fore/background menus in Explorer, Double Commander, and XYPlorer

hi5 commented 2 years ago

This has been added to v1.00 - I haven't added Q-Dir yet as I found it to be unstable (Q-Dir), perhaps this will change at some point with newer versions and more testing.

Suggestions welcome although I won't make any promises to add others.

TooFlow commented 2 years ago

HI，Why Total Commander Can't use F4 to open F4MM (V1.0) at first? The F4MM(V0.97) is OK.

hi5 commented 2 years ago

Works here, just downloaded the script (AutoHotkey version) and the compiled versions (exe, see releases) and they both open notepad with the selected file when I press F4 - check the settings which looks like this by default:

Settings image

and pressing Esc & F4 to bring up the foreground menu image

hi5 commented 2 years ago

Perhaps TC is running as administrator? If so, F4MM also needs to be run as administrator otherwise it won't work.

TooFlow commented 2 years ago

Works here, just downloaded the script (AutoHotkey version) and the compiled versions (exe, see releases) and they both open notepad with the selected file when I press F4 - check the settings which looks like this by default:

Settings image

and pressing Esc & F4 to bring up the foreground menu image

Very Thanks！ That's Useful~~ before this，I often let F4MM open with Windows……HAHA。

skygate2012 commented 1 year ago

Just came to this repo from the forum :) I have to say it's exactly what I had in mind. Albeit with a bit of a learning curve.

Some bugs I've observed, and suggestions:

  1. Currently it's unusable for Explorer for some applications because %p is expanded by sending message to Total Commander. Since the source path is always where the input file(s) is located, maybe just use SplitPath for it and only make a special TC expansion for %t
  2. The path to the executable has to be full, I haven't figured out the purpose of this yet. This is a bit inconvenient when many of the command line binaries are already available in the PATH.
  3. Instead of expanding each variable with StrReplace, I suggest using RegExReplace callouts because it's much more efficient as the string is processed in a single pass. It'll also make the code a lot leaner.
  4. When accessing the Settings dialog via Configure Editors, closing the settings dialog will not trigger the GuiClose event label because Configure Editors is the first Gui, thus the hotkeys are not being re-set. I suggest naming the Settings Gui, and make event labels listen to it instead of the first Gui. For example SettingsGuiClose.
  5. The Add/Modify dialogs for the list editors are not escapable by Esc, which is not convenient for keyboard navigation.
hi5 commented 1 year ago

Thx for the feedback, I'll have a look at these.

hi5 commented 1 year ago

@skygate2012 1,2,4,5 done in v1.2. Didn't bother with 3 as I don't see the problem there at the moment.

skygate2012 commented 1 year ago

@hi5 Thanks! I'll try it out.