Document metrics grid sorting in Discover - #8201
Conversation
Users can sort the metrics grid in Discover, and the page still framed search and breakdown as chart actions. Closes #8200. AI-Assisted: Cursor
Elastic Docs AI PR menuCheck the box to run an AI review for this pull request.
Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team. |
The intro and lead-in were repeating the same capability list. Drop the lead-in so the headings carry the page.
🔍 Preview links for changed docs |
✅ Elastic Docs Style Checker (Vale)No issues found on modified lines! The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
Keep the original intro, but point it at the grid instead of the charts.
Mark Alphabetically as the default and describe Recently explored as order, not as moving metrics.
Drop reload persistence and the extra paragraph; two action examples are enough.
Introduce the list as sorting options instead of a call to action.
Section badges need a heading to render, so 9.5 readers can see this is not available for them.
Page frontmatter already marks the page serverless: ga.
Use a definition list so the badge sits on the term, not in the gap between a bold heading and the list.
alexandra5000
left a comment
alexandra5000
left a comment
There was a problem hiding this comment.
Thank you so much for adding this! I left some comments for your consideration.
| :screenshot: | ||
| ::: | ||
|
|
||
| **Sort the metrics grid** {applies_to}`stack: ga 9.6+` |
There was a problem hiding this comment.
| **Sort the metrics grid** {applies_to}`stack: ga 9.6+` | |
| **Sort the metrics grid** {applies_to}`stack: ga 9.6, serverless: ga` |
Please double-check/confirm, but I think it applies to Serverless too.
There was a problem hiding this comment.
Yes, it does, I didn't keep it because I thought it was redundant. Everything should be available in serverless (at least for us).
For example, in other places I do not have it:
Should we add it also there?
There was a problem hiding this comment.
Hey @miguel-sanchez-elastic, here's some guidance/examples on the cumulative docs strategy. We would generally add the serverless tag just so it's clear that it doesn't only apply to stack. I think this would be especially applicable in your second example because the stack 9.2-9.3 tab wouldn't apply to serverless as serverless is generally at the level of the most recent stack release.
There was a problem hiding this comment.
That being said, I do think the formatting would be:
| **Sort the metrics grid** {applies_to}`stack: ga 9.6+` | |
| **Sort the metrics grid** {applies_to}`stack: ga 9.6+` {applies_to}`serverless: ga` |
There was a problem hiding this comment.
Hi all, I have applied the changes.
Please let me know if this is how you would expect it:
- Sorting:
- The panel that was only mentioning the stack versions:
There was a problem hiding this comment.
@alexandra5000 , do we need your approval as well?
Co-authored-by: Aleksandra Spilkowska <96738481+alexandra5000@users.noreply.github.com>
Co-authored-by: Aleksandra Spilkowska <96738481+alexandra5000@users.noreply.github.com>
Sort and the 9.4+ filter tab also apply to Serverless. Keep 9.2-9.3 as Stack-only because that UI is not on Serverless.
3 participants
Summary
No screenshot in this PR. The most useful one to add later is the Sort dropdown in the metrics toolbar.
Test plan
solutions/observability/infra-and-hosts/discover-metrics.mdapplies_tobadges render on the Sort sectionGenerative AI disclosure