-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[tree][navigator]Tree search #1479
Comments
I've seen this feature only in IntelliJ and like how it works there. |
Did not try it in IntelliJ, isn't it the same as in WebStorm? If no, I will try IntelliJ too. By the way, we are trying to mimic that behavior for sure. |
Yes, it is the same. |
Eclipse:
WebStorm does the following:
I think the WebStorm approach is a good initial pattern to mimic. |
How do they navigate between matches? |
With the Up and Down cursor keys. |
Signed-off-by: Akos Kitta <[email protected]>
+1 it feels nice and simple. |
I just do not know yet, where to overlay the widget for the search term. Any suggestions? |
Signed-off-by: Akos Kitta <[email protected]>
Depends on #1450. I cannot traverse the tree unless I can use the fixed tree iterators. |
Closes #1479. Closes #1493. Signed-off-by: Akos Kitta <[email protected]>
Signed-off-by: Akos Kitta <[email protected]>
CC: @AlexTugarev I am open to any ideas about the search box and its position. We have this so far, but this can be improved for sure: |
Signed-off-by: Akos Kitta <[email protected]>
What about increasing the contrast of this overlay? Maybe add a drop shadow to indicate overlay, as I assume it should stay fixed while scrolling. Another option could be to place is half outside the top border, which would keep the functionality of the main menu. The position on the left hand side would keep the "Search for:" prefix at a fixed position. Shifting that prefix back and forth seems a bit noisy. |
AS Alex mentioned, I find it noisy when you type and see the text moving to the left. I think having it on the left side and half way as Alex mentioned seems a good place to be |
@AlexTugarev, @lmcbout, thanks for the feedback.
Sorry for the confusion, I would go with a minimum width, then it could grow. The probability you would like to search for more than 8-10 characters is quite low.
I like this.
That is good unless you move the navigator to the right, or to the bottom, etc. I have no objections against moving this outside but +1 for keeping this inside. Otherwise, it would look alien compared to other UI elements, isn't it?
Initially, I liked the idea. Now I would say; it's too much. |
That is nice; I am not sure if we want to show the hit count. |
It was more about the drop down effect. |
Yeah, I got it. There is one problem with drop-down effect; when you do this in the browser (I assume, you got the idea from there), you trigger the search with Ctrl/Cmd+F. In our case it should instantly filter whenever you type something, so we cannot have that delay. It should be way faster. |
Another suggestion, |
Not in the context of this task. Please reference the |
Depends on #1517. |
Signed-off-by: Akos Kitta <[email protected]>
Closes #1479. Closes #1493. Signed-off-by: Akos Kitta <[email protected]>
Signed-off-by: Akos Kitta <[email protected]>
Signed-off-by: Akos Kitta <[email protected]>
Closes #1479. Closes #1493. Signed-off-by: Akos Kitta <[email protected]>
Closes #1479. Closes #1493. Signed-off-by: Akos Kitta <[email protected]>
I like it without icons and outside otherwise the navigator looks stuffy and one cannot see all files. IntelliJ also allows drag and drop views to sidebars and they keep the search just above always. I know that it will overlay the menu in our case, but it should be fine since one wants to find something in the navigator not to use menus. Could we try it like that? |
Regarding style cannot it be aligned with the quick search widget? |
Let's try that.
Exactly one file is overlayed.
I do not like that approach but we can try. If it is easy you can decide which one to have. +1 to have it inside. |
Or we can go back to the initial idea and have it after the root node with a min length of 10. So it will not overlay tree nodes still remains inside. |
Signed-off-by: Akos Kitta <[email protected]>
@akosyakov What styles you mean? I cannot find anything special. |
Signed-off-by: Akos Kitta <[email protected]>
If you meant setting the font weight to bold, that is so bad. I have not even noticed it so far :) |
I meant if a search box is the overlay widget similar to the quick command widget it would be nice if they look alike. I did not mean any concrete styles. |
Also reused disposables from the parent widget. Signed-off-by: Akos Kitta <[email protected]>
Closes #1479. Closes #1493. Signed-off-by: Akos Kitta <[email protected]>
Signed-off-by: Akos Kitta <[email protected]>
Closes #1479. Closes #1493. Signed-off-by: Akos Kitta <[email protected]>
Closes #1479. Closes #1493. Signed-off-by: Akos Kitta <[email protected]>
…igator. Closes eclipse-theia#1479. Closes eclipse-theia#1493. Signed-off-by: Akos Kitta <[email protected]>
Must:
Also:
exp
-> then continuelor
-> Then, do we search forlor
orexplor
.Esc
?Update:
Make sure, the following issues are also resolved as part of this task.
Update2:
Q: Does it make sense to support it to all tree widgets in general?
A: It is possible. If we need it we can do this later.
The text was updated successfully, but these errors were encountered: