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

[Discover] Wrong sort order with empty sort URL parameter #97456

Closed
timroes opened this issue Apr 19, 2021 · 1 comment · Fixed by #97434
Closed

[Discover] Wrong sort order with empty sort URL parameter #97456

timroes opened this issue Apr 19, 2021 · 1 comment · Fixed by #97434
Labels
bug Fixes for quality problems that affect the customer experience Feature:Discover Discover Application patch-worthy regression Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@timroes
Copy link
Contributor

timroes commented Apr 19, 2021

Since 7.12 there is a regression when the sort parameter of the application state (URL or saved object) is empty. By default Discover creates saved searches with an empty sort array unless the user sorts by something.

Since 7.12 opening one of those saved searches will lead into the table being sorted by _doc (or I assume the configured tiebreaker field), but still show a sorting indicator on the timestamp column. This is firstly a regression and also very confusing since it's not at all obvious to the user, that they'll need to "resort" the table by that field they already think it's sorted by.

@timroes timroes added bug Fixes for quality problems that affect the customer experience Feature:Discover Discover Application regression Team:Visualizations Visualization editors, elastic-charts and infrastructure patch-worthy labels Apr 19, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:Discover Discover Application patch-worthy regression Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants