Skip to content

fix(heatmap): labels visibility regression#1549

Merged
nickofthyme merged 2 commits intoelastic:masterfrom
markov00:2022_01_14-fix_heatmap_labels_visibility
Jan 14, 2022
Merged

fix(heatmap): labels visibility regression#1549
nickofthyme merged 2 commits intoelastic:masterfrom
markov00:2022_01_14-fix_heatmap_labels_visibility

Conversation

@markov00
Copy link
Collaborator

Summary

This fixes a regression introduced in #1406 where the hidden labels of a heatmap are always shown.

Caused by the removal of this line https://github.com/elastic/elastic-charts/pull/1406/files#diff-c86d69bd8091ee89ee7ab00cab8dc37754a1c7072da64fc3484b0d5b5aeb1734L70

Will fix the issues listed in elastic/kibana#121593

Checklist

  • The proper chart type label has been added (e.g. :xy, :partition)
  • The proper feature labels have been added (e.g. :interactions, :axis)
  • Unit tests have been added or updated to match the most common scenarios

@markov00 markov00 added :chart Chart element related issue regression :heatmap Heatmap/Swimlane chart related issue labels Jan 14, 2022
@markov00 markov00 requested a review from nickofthyme January 14, 2022 11:51
@nickofthyme nickofthyme merged commit 067189d into elastic:master Jan 14, 2022
nickofthyme pushed a commit that referenced this pull request Jan 14, 2022
# [43.0.0](v42.1.0...v43.0.0) (2022-01-14)

### Bug Fixes

* **heatmap:** labels visibility regression ([#1549](#1549)) ([067189d](067189d))
* **xy:** switch default timezone to `local` ([#1544](#1544)) ([1233e69](1233e69))

### BREAKING CHANGES

* **xy:** The time axis labels of a time-series chart configured without the timeZone prop are now rendering the labels with the local browser timezone instead of UTC.
@markov00 markov00 deleted the 2022_01_14-fix_heatmap_labels_visibility branch April 11, 2022 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:chart Chart element related issue :heatmap Heatmap/Swimlane chart related issue regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants