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

New method search function #8

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

Frogli
Copy link

@Frogli Frogli commented May 8, 2020

I did a couple of things. First of the biggest one. To be able to search for and opening methods. See the updated Readme.md for more information. Futher:

  • I added a some functions to remember the window position.
  • A tweak that the input filed only get refocused on window.evTop if the field has not the focus.
  • Refactored to trapping of the kUp and kDown key to a seperate method.

There is one issue with the method searching. I mentioned it in the Readme.md under Known issues. After removing the dot when you are done method searching in a class and want to return to class searching. You have to delete another character to restart the class searching. I did not figure out how to solve this. I expect this has to do with that icFilter is equal to icLastFilter. And I am not sure to reset the class searching.

I hope that you can and will integrate this in your library. Of course feel free to ask or comment on the code. Any suggestions to solve above problem or make the code better are welcome!

Have fun!

h.noppe and others added 10 commits May 2, 2020 22:39
…, show method icon and private method icon
@todo:
* errortrapping for using the F8, F3, etc on methods
* Switching backfrom method search to class search needs an extra backspace
* Autocomplete support (maybe different feature branch)
* Remove the class only complex grid
@todo: fix for starting the class filtering again after deleting the method sepeartor (.)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant