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

RadzenDialog sets height and width to 0px if fast-closing the next opened dialogs #919

New Issue
Closed
opened 2026-01-29 17:46:20 +00:00 by claunia · 3 comments
claunia commented 2026-01-29 17:46:20 +00:00
Owner
Copy Link

Originally created by @Gibrid89 on GitHub (Jul 12, 2023).

*Describe the bug
RadzenDialog sets height and width to 0px if fast-closing the next opened dialogs (reproduce in repo). And in my work project (MAUI) it reproduces when the dialog has only disabled elements and if it is slow closing too.

*To Reproduce in ASP
BlazorApp1.zip
Click the TESTEDIT or TESTNOTEDIT button. Click NEXT to 10 dialogs. Then fast clicks BACK.

*Expected behavior
Save the original height and weight.

*Example:
image

*Desktop

  • OS: Windows 10
  • Browser Chrome 114.0.5735.199
  • Radzen.Blazor 4.13.5
Originally created by @Gibrid89 on GitHub (Jul 12, 2023). *Describe the bug RadzenDialog sets height and width to 0px if fast-closing the next opened dialogs (reproduce in repo). And in my work project (MAUI) it reproduces when the dialog has only disabled elements and if it is slow closing too. *To Reproduce in ASP [BlazorApp1.zip](https://github.com/radzenhq/radzen-blazor/files/12024588/BlazorApp1.zip) Click the TESTEDIT or TESTNOTEDIT button. Click NEXT to 10 dialogs. Then fast clicks BACK. *Expected behavior Save the original height and weight. *Example: ![image](https://github.com/radzenhq/radzen-blazor/assets/34302211/a6d3469c-f767-4709-8b5c-2857bb713454) *Desktop - OS: Windows 10 - Browser Chrome 114.0.5735.199 - Radzen.Blazor 4.13.5
claunia commented 2026-01-29 17:46:21 +00:00
Author
Owner
Copy Link

@Giroxes commented on GitHub (Jul 17, 2023):

Similar error occurs on opening a dialog just after doing a DialogService.Confirm. The last dialog keeps the Confirm dialog dimensions even when options set different width/height

@Giroxes commented on GitHub (Jul 17, 2023): Similar error occurs on opening a dialog just after doing a DialogService.Confirm. The last dialog keeps the Confirm dialog dimensions even when options set different width/height
claunia commented 2026-01-29 17:46:21 +00:00
Author
Owner
Copy Link

@XorZy commented on GitHub (Sep 25, 2023):

I am affected by this as well.
I have not had the time to debug it thoroughly but at a quick glance I think the issue is:
7efb6fc125/Radzen.Blazor/RadzenDialog.razor (L6-L9)
If a dialog is closed and another one is opened just after, the DialogContainer component will get reused, and if the previous dialog instance was resized, then both "width" and "height" will be set, thus the current dialog options will be ignored.

7efb6fc125/Radzen.Blazor/Rendering/DialogContainer.razor (L91-L101)

7efb6fc125/Radzen.Blazor/Rendering/DialogContainer.razor (L156-L162)

A quick workaround is to add a Task.Delay after closing a dialog, so the old DialogContainer is destroyed and a new one is used.

However a proper fix would be welcome considering it should not be too hard to implement.

@XorZy commented on GitHub (Sep 25, 2023): I am affected by this as well. I have not had the time to debug it thoroughly but at a quick glance I think the issue is: https://github.com/radzenhq/radzen-blazor/blob/7efb6fc1250ef23e4ce22ef24248f0dc65a7ca13/Radzen.Blazor/RadzenDialog.razor#L6-L9 If a dialog is closed and another one is opened just after, the DialogContainer component will get reused, and if the previous dialog instance was resized, then both "width" and "height" will be set, thus the current dialog options will be ignored. https://github.com/radzenhq/radzen-blazor/blob/7efb6fc1250ef23e4ce22ef24248f0dc65a7ca13/Radzen.Blazor/Rendering/DialogContainer.razor#L91-L101 https://github.com/radzenhq/radzen-blazor/blob/7efb6fc1250ef23e4ce22ef24248f0dc65a7ca13/Radzen.Blazor/Rendering/DialogContainer.razor#L156-L162 A quick workaround is to add a Task.Delay after closing a dialog, so the old DialogContainer is destroyed and a new one is used. However a proper fix would be welcome considering it should not be too hard to implement.
claunia commented 2026-01-29 17:46:21 +00:00
Author
Owner
Copy Link

@enchev commented on GitHub (Nov 29, 2023):

Fixed c9103bd521

@enchev commented on GitHub (Nov 29, 2023): Fixed https://github.com/radzenhq/radzen-blazor/commit/c9103bd5218d70679885e5910b73b3af3672d414
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
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#919
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?