starred/radzen-blazor
1
0
Fork 0
mirror of https://github.com/radzenhq/radzen-blazor.git synced 2026-02-04 05:35:44 +00:00
Code Issues 1.3k Packages Projects Releases 22 Wiki Activity

[PR #1745] [MERGED] Render chart tooltips in the same way as standard tooltips #2878

New Issue
Open
opened 2026-01-29 18:20:52 +00:00 by claunia · 0 comments
claunia commented 2026-01-29 18:20:52 +00:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/radzenhq/radzen-blazor/pull/1745
Author: @maxbruecken
Created: 10/20/2024
Status: ✅ Merged
Merged: 10/28/2024
Merged by: @akorchev

Base: master ← Head: replace-chart-tooltip-with-radzen-tooltip

📝 Commits (4)

  • 5c2d0f4 Render chart tooltips in the same way as standard tooltips
  • b351d04 Move OpenChartTooltip method to TooltipService to avoid code duplications
  • 542c320 Avoid partially hiding of chart tooltip near top of page
  • b74c448 Make some of the types internal.

📊 Changes

19 files changed (+401 additions, -102 deletions)

View changed files

📝 Radzen.Blazor.Tests/ChartTests.cs (+7 -3)
📝 Radzen.Blazor/CartesianSeries.cs (+21 -19)
📝 Radzen.Blazor/IChartSeries.cs (+8 -4)
📝 Radzen.Blazor/IChartSeriesOverlay.cs (+7 -1)
📝 Radzen.Blazor/RadzenChart.razor (+0 -6)
📝 Radzen.Blazor/RadzenChart.razor.cs (+23 -13)
➕ Radzen.Blazor/RadzenChartTooltip.razor (+184 -0)
📝 Radzen.Blazor/RadzenComponents.razor (+1 -0)
📝 Radzen.Blazor/RadzenSeriesAnnotation.cs (+7 -1)
📝 Radzen.Blazor/RadzenSeriesDataLabels.razor (+7 -1)
📝 Radzen.Blazor/RadzenSeriesTrendLine.razor (+7 -1)
📝 Radzen.Blazor/RadzenSeriesValueLine.razor (+25 -14)
📝 Radzen.Blazor/Rendering/ChartSharedTooltip.razor (+1 -7)
📝 Radzen.Blazor/Rendering/ChartTooltip.razor (+1 -7)
➖ Radzen.Blazor/Rendering/ChartTooltipContainer.razor (+0 -12)
📝 Radzen.Blazor/TooltipService.cs (+38 -1)
📝 Radzen.Blazor/themes/components/blazor/_chart.scss (+36 -6)
📝 Radzen.Blazor/wwwroot/Radzen.Blazor.js (+26 -4)
📝 getting-started.md (+2 -2)

📄 Description

This PR changes rendering of chart tooltip so, that chart tooltips are shown above all another controls, like the standard tooltips. This PR should fix #560.
Unfortunaltely there are some breaking changes:

  • Implementations of IChartSeries and IChartSeriesOverlay in external projects using Radzen must be updated
  • RadzenChartTooltip should be included in e.g. MainLayout.razor and ChartTooltipService should bei injected. No changes are necessary if RadzenComponents.razor and .AddRadzenComponents() are already used

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/radzenhq/radzen-blazor/pull/1745 **Author:** [@maxbruecken](https://github.com/maxbruecken) **Created:** 10/20/2024 **Status:** ✅ Merged **Merged:** 10/28/2024 **Merged by:** [@akorchev](https://github.com/akorchev) **Base:** `master` ← **Head:** `replace-chart-tooltip-with-radzen-tooltip` --- ### 📝 Commits (4) - [`5c2d0f4`](https://github.com/radzenhq/radzen-blazor/commit/5c2d0f4b1dc6fdadbc3a2a27afc707ba07504ad0) Render chart tooltips in the same way as standard tooltips - [`b351d04`](https://github.com/radzenhq/radzen-blazor/commit/b351d0479ec4b93327af8921b465df5ca0b5d871) Move OpenChartTooltip method to TooltipService to avoid code duplications - [`542c320`](https://github.com/radzenhq/radzen-blazor/commit/542c32088fad4707c8887f0cd4157ff564f6c551) Avoid partially hiding of chart tooltip near top of page - [`b74c448`](https://github.com/radzenhq/radzen-blazor/commit/b74c44855615d755cedbfcbb95653a7281f6567b) Make some of the types internal. ### 📊 Changes **19 files changed** (+401 additions, -102 deletions) <details> <summary>View changed files</summary> 📝 `Radzen.Blazor.Tests/ChartTests.cs` (+7 -3) 📝 `Radzen.Blazor/CartesianSeries.cs` (+21 -19) 📝 `Radzen.Blazor/IChartSeries.cs` (+8 -4) 📝 `Radzen.Blazor/IChartSeriesOverlay.cs` (+7 -1) 📝 `Radzen.Blazor/RadzenChart.razor` (+0 -6) 📝 `Radzen.Blazor/RadzenChart.razor.cs` (+23 -13) ➕ `Radzen.Blazor/RadzenChartTooltip.razor` (+184 -0) 📝 `Radzen.Blazor/RadzenComponents.razor` (+1 -0) 📝 `Radzen.Blazor/RadzenSeriesAnnotation.cs` (+7 -1) 📝 `Radzen.Blazor/RadzenSeriesDataLabels.razor` (+7 -1) 📝 `Radzen.Blazor/RadzenSeriesTrendLine.razor` (+7 -1) 📝 `Radzen.Blazor/RadzenSeriesValueLine.razor` (+25 -14) 📝 `Radzen.Blazor/Rendering/ChartSharedTooltip.razor` (+1 -7) 📝 `Radzen.Blazor/Rendering/ChartTooltip.razor` (+1 -7) ➖ `Radzen.Blazor/Rendering/ChartTooltipContainer.razor` (+0 -12) 📝 `Radzen.Blazor/TooltipService.cs` (+38 -1) 📝 `Radzen.Blazor/themes/components/blazor/_chart.scss` (+36 -6) 📝 `Radzen.Blazor/wwwroot/Radzen.Blazor.js` (+26 -4) 📝 `getting-started.md` (+2 -2) </details> ### 📄 Description This PR changes rendering of chart tooltip so, that chart tooltips are shown above all another controls, like the standard tooltips. This PR should fix #560. Unfortunaltely there are some breaking changes: - Implementations of IChartSeries and IChartSeriesOverlay in external projects using Radzen must be updated - RadzenChartTooltip should be included in e.g. MainLayout.razor and ChartTooltipService should bei injected. No changes are necessary if RadzenComponents.razor and .AddRadzenComponents() are already used --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 18:20:52 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
enhancement
good first issue
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claunia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/radzen-blazor#2878
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?