Skip to content

Conversation

onesounds
Copy link
Contributor

@onesounds onesounds commented Oct 30, 2021

@onesounds onesounds changed the title File manager Default File manager Oct 30, 2021
@taooceros
Copy link
Member

I will merge #767 first for fody

@onesounds
Copy link
Contributor Author

onesounds commented Nov 6, 2021

add arguments of files / total commander / directory opus.
It seems that it cannot be tested with a plug-in yet, so it was tested with a command line.
I don't know how to run rx explorer as a command line.

References
https://resource.dopus.com/t/arguments-or-parameters-for-windows-shortcuts/5099
https://www.voidtools.com/forum/viewtopic.php?t=6617
https://www.ghisler.ch/wiki/index.php/Command_line_parameters
files-community/Files#6558

  • This item is likely to be added/changed in the future. When I tested, there was a problem that ended when the data changed slightly. It seems that there is a need for a way to be compatible with the old version of data or reset if the data does not fit. The web search plug-in has the same problem, so this kind of setting seems to require a separate solution

  • I think we should make it impossible to put an argument as an empty item. If you press "done" in the blank item state, you can automatically enter and save "%d", or if the textbox length is 0, you can enter "%d" from the beginning.

  • For a detailed description of the argument setting, let's add a hyperlink. I think we can fill out a document and link it.

@taooceros
Copy link
Member

Done a rebase to clean some history. Nothing actually change in code

@onesounds onesounds marked this pull request as ready for review November 10, 2021 21:21
jjw24
jjw24 previously approved these changes Nov 10, 2021
@jjw24 jjw24 added the enhancement New feature or request label Nov 10, 2021
@jjw24 jjw24 added this to the 1.9.0 milestone Nov 10, 2021
@jjw24 jjw24 enabled auto-merge November 10, 2021 21:50
jjw24
jjw24 previously approved these changes Nov 10, 2021
@jjw24
Copy link
Member

jjw24 commented Nov 11, 2021

@onesounds got conflict need to resolve for this one

@onesounds
Copy link
Contributor Author

@onesounds got conflict need to resolve for this one

Done.

@onesounds onesounds disabled auto-merge November 11, 2021 12:24
@jjw24 jjw24 enabled auto-merge November 11, 2021 20:32
@jjw24 jjw24 merged commit b7d5f4f into Flow-Launcher:dev Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants