Skip to content

[8.19] [Discover] Lens config extension point and hiding x-axis tick labels in patterns profile (#225571)#226056

Merged
jughosta merged 2 commits intoelastic:8.19from
jughosta:backport/8.19/pr-225571
Jul 1, 2025
Merged

[8.19] [Discover] Lens config extension point and hiding x-axis tick labels in patterns profile (#225571)#226056
jughosta merged 2 commits intoelastic:8.19from
jughosta:backport/8.19/pr-225571

Conversation

@jughosta
Copy link
Contributor

@jughosta jughosta commented Jul 1, 2025

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…in patterns profile (elastic#225571)

## Summary

This PR introduces a `getModifiedVisAttributes` extension point,
allowing profiles to customize the default Lens chart configuration.
Additionally, it updates the patterns data source profile to make use of
the extension point in order to hide x-axis tick labels in the XY chart.
It also stops hiding the chart by default for the patterns profile.

### Checklist

- [ ] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [ ]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [ ] If a plugin configuration key changed, check if it needs to be
allowlisted in the cloud and added to the [docker
list](https://github.com/elastic/kibana/blob/main/src/dev/build/tasks/os_packages/docker_generator/resources/base/bin/kibana-docker)
- [ ] This was checked for breaking HTTP API changes, and any breaking
changes have been approved by the breaking-change committee. The
`release_note:breaking` label should be applied in these situations.
- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

---------

Co-authored-by: James Gowdy <jgowdy@elastic.co>
(cherry picked from commit b53844f)

# Conflicts:
#	src/platform/packages/shared/kbn-unified-histogram/hooks/use_unified_histogram.ts
#	src/platform/plugins/shared/discover/public/application/main/components/layout/use_discover_histogram.test.tsx
#	src/platform/plugins/shared/discover/public/application/main/components/layout/use_discover_histogram.ts
#	src/platform/plugins/shared/discover/public/context_awareness/__mocks__/context_awareness.tsx
@jughosta jughosta requested a review from kibanamachine as a code owner July 1, 2025 16:42
@jughosta jughosta added the backport This PR is a backport of another PR label Jul 1, 2025
@jughosta jughosta enabled auto-merge (squash) July 1, 2025 16:42
@jughosta jughosta disabled auto-merge July 1, 2025 16:42
@jughosta jughosta requested a review from jgowdyelastic July 1, 2025 16:42
Copy link
Member

@jgowdyelastic jgowdyelastic left a comment

Choose a reason for hiding this comment

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

Tested, LGTM

@elasticmachine
Copy link
Contributor

elasticmachine commented Jul 1, 2025

💔 Build Failed

Failed CI Steps

History

@jughosta jughosta enabled auto-merge (squash) July 1, 2025 17:52
@jughosta jughosta merged commit e96f119 into elastic:8.19 Jul 1, 2025
8 checks passed
@jughosta jughosta deleted the backport/8.19/pr-225571 branch July 2, 2025 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants