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

Find pane issues and details #13657

Closed
4 tasks
dalthviz opened this issue Aug 31, 2020 · 2 comments · Fixed by #13675
Closed
4 tasks

Find pane issues and details #13657

dalthviz opened this issue Aug 31, 2020 · 2 comments · Fixed by #13675

Comments

@dalthviz
Copy link
Member

Some issues and details that needed to be worked out in the Find pane

  • The hamburger menu is not the same as in 4.x (some actions are not being listed)
  • The label of the directory was changed (from Search in: to Location).
  • A label that says Search was added to the combobox used to type search terms but the button already says Search on the right (so it is kind of redundant).
  • In the Editor having a word selected and hitting Cmd + Shift + F/Ctrl + Shift + F doesn’t automatically start a search.
@dalthviz
Copy link
Member Author

@goanpeca could you look into the listed elements? Thanks!

@dalthviz dalthviz assigned dalthviz and goanpeca and unassigned dalthviz Aug 31, 2020
@goanpeca
Copy link
Member

goanpeca commented Aug 31, 2020

Thanks for the feedback @dalthviz, will work on the fixes.

However for this one:

In the Editor having a word selected and hitting Cmd + Shift + F/Ctrl + Shift + F doesn’t automatically start a search.

It cannot be fixed to behave as 4.x, since on 4.x it is merging 2 different actions under the same shortcut and the new API does not allow for these hacks to work. It is not a matter of the new API not being able to do something and more of a case that things in 4.x (and before since this probably goes back to 2.x ...) being done in a peculiar way, to save ... something.

On 4.x:

  • There is the Switch to Find in Files and that is set Cmd + Shift + F by default.
    image

image

  • Another completely different action that is added to the Search Menu is taking the selected text from the Editor, pasting on the Find in Files plugin, showing it if it hidden and then running the search.
    image

  • This completely different action is also using the same Cmd + Shift + F shortcut.

On master:

  • There is the Switch to Find in Files and that is set Cmd + Shift + F by default. Triggering this action will show the pane, as expected
  • There is the Find in Files action that will show the plugin if not visible and start running the find in files pluginif there is text in it. It is not copying the text and pasting it on the plugin (This I will fix)
  • This separate action is available now on the shortcut preferences and can be configured, but cannot use the same shortcut.

Action points:

  1. I can have the menu do the expected action, but not use the same shortcut as another action (Switch to Find in files).
  2. I can add a default shortcut (whatever is decided?) or this action will not have a shortcut by default. Note that currently this action is not exposed in the shortcut preferences (it is a matter of changing register_shortcut=True when creating the action).
  3. I can fix the selected text behavior so it does the same thing on master

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

Successfully merging a pull request may close this issue.

3 participants