[PR #976] [MERGED] Improved chart tooltips #2438

Open
opened 2026-01-29 18:18:49 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/radzenhq/radzen-blazor/pull/976
Author: @stlufred
Created: 5/13/2023
Status: Merged
Merged: 5/23/2023
Merged by: @akorchev

Base: masterHead: master


📝 Commits (5)

  • ba8ff59 Improved chart tooltips
  • 7a878f0 Moved caching into the Category method
  • da5a9e4 Reset categoryPropertyCache in SetParametersAsync.
  • fbe11a2 Revert tooltips to always show above
  • 3dbd944 Update _chart.scss

📊 Changes

9 files changed (+113 additions, -51 deletions)

View changed files

📝 Radzen.Blazor.Tests/ChartTests.cs (+12 -2)
📝 Radzen.Blazor/CartesianSeries.cs (+31 -10)
📝 Radzen.Blazor/IChartSeries.cs (+3 -2)
📝 Radzen.Blazor/RadzenBarSeries.razor.cs (+5 -5)
📝 Radzen.Blazor/RadzenChart.razor.cs (+47 -17)
📝 Radzen.Blazor/RadzenColumnSeries.razor.cs (+4 -4)
📝 Radzen.Blazor/RadzenPieSeries.razor.cs (+3 -3)
📝 Radzen.Blazor/RadzenStackedBarSeries.razor.cs (+4 -4)
📝 Radzen.Blazor/RadzenStackedColumnSeries.razor.cs (+4 -4)

📄 Description

  • Show closest tooltip in line chart (instead of the first close enough)
  • Improved speed (tooltip performance unit test execution time reduced from 4.3 to 1.7 seconds)
  • Show it over or under to greatly reduce chances of tooltip being offscreen

🔄 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/976 **Author:** [@stlufred](https://github.com/stlufred) **Created:** 5/13/2023 **Status:** ✅ Merged **Merged:** 5/23/2023 **Merged by:** [@akorchev](https://github.com/akorchev) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (5) - [`ba8ff59`](https://github.com/radzenhq/radzen-blazor/commit/ba8ff5930de587b2ff8f249dbd321ea315e6b2bd) Improved chart tooltips - [`7a878f0`](https://github.com/radzenhq/radzen-blazor/commit/7a878f0f1442d87de32cb4c5c2ea3573ecdbcdd2) Moved caching into the Category method - [`da5a9e4`](https://github.com/radzenhq/radzen-blazor/commit/da5a9e4095392a79e9e1eed7cfd0c3bef8c6bafa) Reset categoryPropertyCache in SetParametersAsync. - [`fbe11a2`](https://github.com/radzenhq/radzen-blazor/commit/fbe11a23747f43f1114566c66056608d4afaf749) Revert tooltips to always show above - [`3dbd944`](https://github.com/radzenhq/radzen-blazor/commit/3dbd9440c65c899a1578184ef7ed6580cc4b6a20) Update _chart.scss ### 📊 Changes **9 files changed** (+113 additions, -51 deletions) <details> <summary>View changed files</summary> 📝 `Radzen.Blazor.Tests/ChartTests.cs` (+12 -2) 📝 `Radzen.Blazor/CartesianSeries.cs` (+31 -10) 📝 `Radzen.Blazor/IChartSeries.cs` (+3 -2) 📝 `Radzen.Blazor/RadzenBarSeries.razor.cs` (+5 -5) 📝 `Radzen.Blazor/RadzenChart.razor.cs` (+47 -17) 📝 `Radzen.Blazor/RadzenColumnSeries.razor.cs` (+4 -4) 📝 `Radzen.Blazor/RadzenPieSeries.razor.cs` (+3 -3) 📝 `Radzen.Blazor/RadzenStackedBarSeries.razor.cs` (+4 -4) 📝 `Radzen.Blazor/RadzenStackedColumnSeries.razor.cs` (+4 -4) </details> ### 📄 Description - Show closest tooltip in line chart (instead of the first close enough) - Improved speed (tooltip performance unit test execution time reduced from 4.3 to 1.7 seconds) - Show it over or under to greatly reduce chances of tooltip being offscreen --- <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:18:49 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/radzen-blazor#2438