Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

XYChart2: Fix tooltip mode #87147

Merged
merged 2 commits into from
Apr 30, 2024
Merged

XYChart2: Fix tooltip mode #87147

merged 2 commits into from
Apr 30, 2024

Conversation

adela-almasan
Copy link
Contributor

Fixes: tooltip visibility after switching between hidden/single mode; series name

xy_tooltip.mov

Fixes #85368

Please check that:

  • It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

@adela-almasan adela-almasan added area/frontend area/panel/xychart no-changelog Skip including change in changelog/release notes backport v11.0.x Mark PR for automatic backport to v11.0.x labels Apr 30, 2024
@adela-almasan adela-almasan added this to the 11.1.x milestone Apr 30, 2024
@adela-almasan adela-almasan self-assigned this Apr 30, 2024
@adela-almasan adela-almasan requested a review from a team as a code owner April 30, 2024 16:53
@adela-almasan adela-almasan requested review from leeoniya and baldm0mma and removed request for a team April 30, 2024 16:53
@leeoniya leeoniya changed the title XYChart: Fix tooltip mode XYChart2: Fix tooltip mode Apr 30, 2024
Copy link
Contributor

This PR must be merged before a backport PR will be created.

@@ -24,7 +24,7 @@ export interface Props {
}

function stripSeriesName(fieldName: string, seriesName: string) {
if (fieldName.includes(' ')) {
if (fieldName !== seriesName && fieldName.includes(' ')) {
Copy link
Contributor

Choose a reason for hiding this comment

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

(this is fixing a regression from #86739)

@adela-almasan adela-almasan merged commit 4312f2a into main Apr 30, 2024
33 checks passed
@adela-almasan adela-almasan deleted the xy_tooltip_mode branch April 30, 2024 18:19
@grafana grafana deleted a comment from grafana-delivery-bot bot Apr 30, 2024
grafana-delivery-bot bot pushed a commit that referenced this pull request Apr 30, 2024
(cherry picked from commit 4312f2a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend area/panel/xychart backport v11.0.x Mark PR for automatic backport to v11.0.x no-changelog Skip including change in changelog/release notes type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

XYChart2: Tooltip mode switching not working
2 participants