-
-
Notifications
You must be signed in to change notification settings - Fork 455
Explorer plugin: Support AutoCompleteText Field #863
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
Explorer plugin: Support AutoCompleteText Field #863
Conversation
Fix misaligned suggestion text
Co-authored-by: Jeremy Wu <[email protected]>
Co-authored-by: Jeremy Wu <[email protected]>
|
Need to support a custom action keyword and should be good to go. |
|
what do you mean? in this pr? |
Yeah, adding as we speak... If the user has a custom keyword we need to append it to autocomplete |
|
oh right, yes. |
|
Moved some stuff around so we don't repeat too much logic... Come to think of it... should Enter be changed to open the directories? |
|
no, enter goes into the directory, i am actually wanting to change Everything plugin's enter functionality. |
| Keyword = Settings.SearchActionKeywordEnabled ? Settings.SearchActionKeyword : Settings.PathSearchActionKeyword; | ||
| Keyword = Keyword.ToString() == Query.GlobalPluginWildcardSign ? string.Empty : Keyword + " "; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i see why you did it this way, but because the action keyword could change during the runtime of the app, this means Keyword is only set once during Init and with a new string value. We need to get this dynamically because could change anytime.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i am working on tweaking this, will push up when done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In my testing it worked even when hotkey changed.
Edit: Never mind you're right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done, please test it now
|
f741420 is working thanks! |
Adds Auto Complete support to the file explorer plugin