Skip to content

Consolidate duplicate "Access Lens" and "Save" steps into shared snippets across all 13 chart pages#7020

Merged
florent-leborgne merged 5 commits into
mainfrom
copilot/docs-fix-path-explore-analyze
Jun 25, 2026
Merged

Consolidate duplicate "Access Lens" and "Save" steps into shared snippets across all 13 chart pages#7020
florent-leborgne merged 5 commits into
mainfrom
copilot/docs-fix-path-explore-analyze

Conversation

Copilot AI commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

All 13 Lens chart pages repeated identical (or near-identical) "Access Lens" and "Save the chart" step content inline, diverging in minor wording over time. This follows the existing lens-prerequisites.md snippet pattern already in use on these pages.

New snippets

  • explore-analyze/_snippets/access-lens.md — generic "Access Lens" step body (only variation was chart type name, replaced with "the visualization")
  • explore-analyze/_snippets/save-visualization.md — canonical "Save" step body with consistent wording

All 13 chart pages updated

Inline content replaced with :::{include} directives:

:::::{step} Access Lens
:::{include} ../../_snippets/access-lens.md
:::
:::::

:::::{step} Save the chart
:::{include} ../../_snippets/save-visualization.md
:::
:::::

Wording standardized

  • "offers you to add""lets you add" (gauge, heat-map)
  • "be able to add it to other dashboards later""reuse it later" (bar, metric, tables)
  • line-charts.md truncated save instructions restored to full canonical text
  • Fixed double-period typo in metric-charts.md

…ippets

- Create explore-analyze/_snippets/access-lens.md with generic Access Lens step content
- Create explore-analyze/_snippets/save-visualization.md with canonical Save step content
- Replace duplicate step bodies in all 13 chart pages with include directives
- Fix double-period typo in metric-charts.md
- Standardize wording variations (e.g. "offers you to add" → "lets you add", "be able to add it to other dashboards later" → "reuse it later")

Follows the existing lens-prerequisites.md snippet pattern already used across all chart pages.

Closes #6884
Copilot AI changed the title [WIP] Fix duplicate content in charts documentation Consolidate duplicate "Access Lens" and "Save" steps into shared snippets across all 13 chart pages Jun 19, 2026
Copilot AI requested a review from florent-leborgne June 19, 2026 09:46
Comment thread explore-analyze/_snippets/save-visualization.md Outdated
@github-actions

Copy link
Copy Markdown
Contributor

Elastic Docs AI PR menu

Check the box to run an AI review for this pull request.

  • Review docs changes (docs-review). Status: not started.

Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team.

@github-actions

Copy link
Copy Markdown
Contributor

✅ 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.

@florent-leborgne

Copy link
Copy Markdown
Member

I assigned this task to AI, reviewed it, and made any required changes.

Ready for formal review.

elky says go!

@florent-leborgne florent-leborgne marked this pull request as ready for review June 19, 2026 10:22
@florent-leborgne florent-leborgne requested review from a team as code owners June 19, 2026 10:22

@leemthompo leemthompo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @copilot!

@florent-leborgne florent-leborgne enabled auto-merge (squash) June 25, 2026 14:58
@florent-leborgne florent-leborgne merged commit bcbb7e3 into main Jun 25, 2026
7 of 8 checks passed
@florent-leborgne florent-leborgne deleted the copilot/docs-fix-path-explore-analyze branch June 25, 2026 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs fix — coherence: path /explore-analyze/visualize/charts — 2 findings

3 participants