-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Inline editing of position and query #3308
Comments
|
Switching to Bootstrap 4 is certainly an option, there were some issues with RTL support last time I looked, see #2916. For the search field, I'd really like to make it more flexible to allow building more complex queries easily. I've mentioned some examples (GitLab and Sentry) in #3063 (comment) and #3063 (comment) |
@nijel Great! In that case, won't it make more sense to separate the search field from the pagination group of buttons on the right sounds? And we can just keep the tab in pagination to switch between different filters of |
Yes that makes sense. UX wise still many of parts are results of myself not being comfortable in CSS and JS to do it way it would look best, so it ended up using default components present in bootstrap. |
See also #3742 |
I was proposing something similar. |
I think we could use something GitLab like as described in #3063 (comment). Example:
|
Agree with this. I feel first we should go with separting this 3 parts like in gitlab and then iteratively focus on adding multiple elements (or query terms) in the query box like gitlab allows. |
Thinking more about this, the final state should be following (I'm not saying we should reach it in single step, but just to define the direction we're heading):
The basic look should be following (it lacks the navigation): Screenshots from Sentry to show how it behaves: Recent queries shown on focus to text edit: Autocompletion on tag: Autocompletion on value: |
@nijel sounds great to me. I am gonna proceed with it iteratively building first the UI and then improving on the different features. |
* Query Field: Separates query field from the pagination part Separates the query search field from the pagination part and adds filters and sorting options in UI * Query Field: Adds sorting feature from the query field dropdown * Refactors code for sorting and filtering * Autosort and auto filter queries on selecting dropdown * Adds selenium tests and more sort options * Adds fixes for some accessibility issues related to input * Refactor codes to make more readable * Fix selenium tests - add delay before screenshots to avoid capturing transitions - revert to original search for other tests * Adds sorting direction feature with suitable icons * Shows dropdown labels and enables autosearch in page with results * Use bootstrap button group for sorting * Use default autosizing width of button for sort dropdown * Fixes button display in RTL * Adds sort_by to SearchForm along with the direction * Uses a separate class to identify autosubmit * Adds query search field in zen pages * Adds check to see if the current page has a sort field and a sort value set Issue #3308, #3710 Co-authored-by: Michal Čihař <[email protected]>
Query editing landed in #3766, the position needs to be done. |
Thank you for your report, the issue you have reported has just been fixed.
|
Is your feature request related to a problem? Please describe.
With improved search based on query string instead of big forms, it seems better to allow editing the query string and position inline instead of poping up a dialog.
Describe the solution you'd like
Click on search term or position and input box appears in the position, enter submits it, escape cancels the edit.
Describe alternatives you've considered
At least proper popup placement would be great.
Additional context

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: