Popover: add Legacy mode (uses old MudPopoverService)#7497
Popover: add Legacy mode (uses old MudPopoverService)#7497ScarletKuro merged 3 commits intoMudBlazor:devfrom
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## dev #7497 +/- ##
==========================================
- Coverage 90.61% 90.57% -0.04%
==========================================
Files 427 427
Lines 15198 15210 +12
==========================================
+ Hits 13772 13777 +5
- Misses 1426 1433 +7
☔ View full report in Codecov by Sentry. |
|
Interesting, I'd also love to know what it is that causes that initial delay. Any hunches? In any case, we'll have to be able to reproduce the problem, otherwise it would certainly be hard to test if the issue is gone after making changes or we'd fully rely on the OP of the bug report. |
I only see one difference currently: |
Yeah, that's a big problem. OP says it happens only in their AWS cloud, but i wonder if that happens only on their specific codebase, because I'm sure other BSS folks would report this long time ago. |
Description
See for context #7434
TL;DR - A customer reported an issue with Blazor Server Side performance with the popovers (only the initial load), and we identified that PR #6953 was the cause. I intend to investigate and address the root cause when I have more time. Meanwhile, to provide a quick solution, we decided to introduce a backward compatibility option that allows users to easily switch to the old popover service (that contains it's own issues tho).
To enable the old popover:
How Has This Been Tested?
Manually, #7434 (reply in thread)
Types of changes
Checklist:
dev).