Skip to content

fix: disable cursor line and hover markers for heatmap series#211

Merged
pan-kot merged 2 commits intocloudscape-design:mainfrom
ryugaraj:main
Apr 27, 2026
Merged

fix: disable cursor line and hover markers for heatmap series#211
pan-kot merged 2 commits intocloudscape-design:mainfrom
ryugaraj:main

Conversation

@ryugaraj
Copy link
Copy Markdown
Contributor

@ryugaraj ryugaraj commented Apr 23, 2026

Heatmap cells don't benefit from the vertical cursor line or point markers shown on hover. Exclude heatmap from both, matching the existing behavior for column/errorbar series.

Renamed hasColumnSeries to hideCursorLine for clarity.

Description

Removed cursor line and the highlighted "datapoint" on hover

How has this been tested?

Tested with Heatmap in CW and Heatmap demo page in CoreChart

Screenshot 2026-04-27 at 17 12 10

Correctness

  • Changes include appropriate documentation updates.
  • Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
  • Changes do not include unsupported browser features, see CONTRIBUTING.md.
  • Changes were manually tested for accessibility, see accessibility guidelines.

Security

Testing

  • Changes are covered with new/existing unit tests?
  • Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ryugaraj ryugaraj marked this pull request as ready for review April 23, 2026 10:26
@ryugaraj ryugaraj requested a review from a team as a code owner April 23, 2026 10:26
@ryugaraj ryugaraj requested review from taheramr and removed request for a team April 23, 2026 10:26
Raj Jaiswal added 2 commits April 27, 2026 17:53
Heatmap cells don't benefit from the vertical cursor line or point
markers shown on hover. Exclude heatmap from both, matching the
existing behavior for column/errorbar series.

Renamed hasColumnSeries to hideCursorLine for clarity.
@pan-kot pan-kot enabled auto-merge April 27, 2026 17:00
@pan-kot pan-kot added this pull request to the merge queue Apr 27, 2026
Merged via the queue into cloudscape-design:main with commit 01c7ecf Apr 27, 2026
42 of 44 checks passed
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.

2 participants