Skip to content

[BUGFIX]: Remove selectedMode from timeseries series config#722

Merged
Nexucis merged 1 commit into
mainfrom
ibakshay/fix-timeseries-hover-stack
Jul 16, 2026
Merged

[BUGFIX]: Remove selectedMode from timeseries series config#722
Nexucis merged 1 commit into
mainfrom
ibakshay/fix-timeseries-hover-stack

Conversation

@ibakshay

@ibakshay ibakshay commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Related to perses/shared#195

Description

Part of the tooltip hover fix for stacked and grouped charts.
Removes selectedMode: 'single' and its properties from the timeseries series config. These options enabled eCharts' built-in selection state, which conflicted with the new emphasis/downplay logic and caused highlight markers on series that were no longer hovered.

Screenshots

Screen.Recording.2026-07-03.at.23.53.38.mov

Checklist

  • Pull request has a descriptive title and context useful to a reviewer.
  • Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the
    following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, DOC,IGNORE.
  • All commits have DCO signoffs.

UI Changes

  • Changes that impact the UI include screenshots and/or screencasts of the relevant changes.
  • Code follows the UI guidelines.

Signed-off-by: Akshay Iyyadurai Balasundaram <akshay.iyyadurai.balasundaram@sap.com>
@ibakshay
ibakshay requested a review from a team as a code owner July 3, 2026 18:26
@ibakshay
ibakshay requested review from shahrokni and removed request for a team July 3, 2026 18:26
@ibakshay ibakshay changed the title BUGFIX: Remove selectedMode from timeseries series config [BUGFIX]: Remove selectedMode from timeseries series config Jul 4, 2026
@Nexucis
Nexucis added this pull request to the merge queue Jul 16, 2026
Merged via the queue into main with commit 2124dff Jul 16, 2026
15 checks passed
@Nexucis
Nexucis deleted the ibakshay/fix-timeseries-hover-stack branch July 16, 2026 13:01
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