-
Notifications
You must be signed in to change notification settings - Fork 193
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
Search dialog should become a search view which is not blocking #883
Comments
Instead of a complete new view, maybe this could be incooperated into the "Search Results"? |
@laeubi also a valid option but most likely more complex as the Search view can also be filled via other means (IIRC). |
Yeah but maybe it would make sense to have a "centralized" approach then to make those option available (non blocking), e.g. what I can think of is to have a textfield for the search, then on the left i can choose the type (E.g. File Search, Git Search, Java Search...) and on the right a "configure" button that opens a flyout to show the other options of the search (like when you minimized a view). Just a rough idea... having just another view might be confusing as actually they are closely related. |
Whoever decides to work on this can select the approach. :-) |
What about just making it non-modal and making sure you can't open it more than once (just like the Find dialog). I don't like the view idea at all. |
That would be already an improvement, like I said: Whoever decides to work on this can select the approach. :-)
Why? |
I'm also not fan of a view in order to configure things. Dialog are semantically good to capture a user request, a view is good to get a view of something. Using a view to capture user input seems awkward to me. From here, I agree with @merks that the issues listed in the initial comment don't seem to mandate a view, and would be solved by simply making the dialog non-modal. |
As i change the search text much more frequent then the other options i propose to just add a small input field for that text. |
The blocking search dialog has certain disadvantages:
I suggest we move this dialog to a part, this way the user can still detach it if desired.
In theory it should be easy to do this change, we just replace the dialog with a part implementation.
The text was updated successfully, but these errors were encountered: