Skip to content

[8.x] [Lens][Datatable] Fix color by value for Last value array mode (#213917)#216117

Merged
nickofthyme merged 1 commit intoelastic:8.xfrom
nickofthyme:backport/8.x/pr-213917
Mar 28, 2025
Merged

[8.x] [Lens][Datatable] Fix color by value for Last value array mode (#213917)#216117
nickofthyme merged 1 commit intoelastic:8.xfrom
nickofthyme:backport/8.x/pr-213917

Conversation

@nickofthyme
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…ic#213917)

## Summary

Fixes the table side of elastic#188263

The [fix used for
`Metric`](elastic#209110) has been
generalized and re-used for the datatable visualization.

![color_by_value_table](https://github.com/user-attachments/assets/b347dba2-24d7-4233-8c0c-3236f5212f35)

### Checklist

- [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

(cherry picked from commit 0b9df09)

# Conflicts:
#	x-pack/platform/plugins/shared/lens/public/shared_components/coloring/utils.ts
#	x-pack/platform/plugins/shared/lens/public/visualizations/datatable/components/dimension_editor.test.tsx
#	x-pack/platform/plugins/shared/lens/public/visualizations/datatable/visualization.tsx
#	x-pack/platform/plugins/shared/lens/public/visualizations/metric/dimension_editor.tsx
@nickofthyme nickofthyme added the backport This PR is a backport of another PR label Mar 27, 2025
@nickofthyme nickofthyme enabled auto-merge (squash) March 27, 2025 00:58
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
lens 1500 1499 -1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
lens 1.5MB 1.5MB +104.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
lens 58.8KB 58.8KB +14.0B

@nickofthyme nickofthyme requested a review from dej611 March 27, 2025 22:55
@nickofthyme nickofthyme merged commit c058bbe into elastic:8.x Mar 28, 2025
11 checks passed
@nickofthyme nickofthyme deleted the backport/8.x/pr-213917 branch March 28, 2025 18:24
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