Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
hi5 opened this issue May 9, 2021 · 9 comments
Open

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

hi5 opened this issue May 9, 2021 · 9 comments
Assignees

Comments

@hi5
Copy link
Owner

hi5 commented May 9, 2021

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

@hi5 hi5 self-assigned this May 9, 2021
@hi5 hi5 added the enhancement label May 9, 2021
@hi5
Copy link
Owner Author

hi5 commented Sep 26, 2022

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
Copy link

TooFlow commented Oct 2, 2022

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

@hi5
Copy link
Owner Author

hi5 commented Oct 3, 2022

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
Copy link
Owner Author

hi5 commented Oct 3, 2022

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

@TooFlow
Copy link

TooFlow commented Oct 7, 2022

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
Copy link

skygate2012 commented Jun 2, 2023

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
Copy link
Owner Author

hi5 commented Jun 3, 2023

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

@hi5
Copy link
Owner Author

hi5 commented Oct 30, 2023

@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
Copy link

@hi5 Thanks! I'll try it out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants