From 0623c63f782010ce89b231b7b2656bb5006f0ace Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 26 Sep 2025 04:07:02 +0000
Subject: [PATCH 1/5] Initial plan


From 42c41361d01c12745d41157d84028a336c87d3e6 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 26 Sep 2025 04:19:36 +0000
Subject: [PATCH 2/5] Implemented fix for multiple dialog scrolling issue

Co-authored-by: ArgoZhang <22001478+ArgoZhang@users.noreply.github.com>
---
 .../Components/Pages/TestScrollDialog.razor   | 134 ++++++++++++++++++
 .../Components/Modal/Modal.razor.scss         |  17 +++
 2 files changed, 151 insertions(+)
 create mode 100644 src/BootstrapBlazor.Server/Components/Pages/TestScrollDialog.razor

diff --git a/src/BootstrapBlazor.Server/Components/Pages/TestScrollDialog.razor b/src/BootstrapBlazor.Server/Components/Pages/TestScrollDialog.razor
new file mode 100644
index 00000000000..04635390d86
--- /dev/null
+++ b/src/BootstrapBlazor.Server/Components/Pages/TestScrollDialog.razor
@@ -0,0 +1,134 @@
+@page "/test-scroll-dialog"
+@using BootstrapBlazor.Server.Components.Components
+@inject DialogService DialogService
+
+<PageTitle>Test Scroll Dialog</PageTitle>
+
+<h1>Test Multiple Dialog Scrolling</h1>
+
+<p>This page tests the multiple dialog scrolling functionality.</p>
+
+<div class="mb-3">
+    <Button Text="Open First Dialog" OnClick="@OpenFirstDialog" />
+</div>
+
+@code {
+    private async Task OpenFirstDialog()
+    {
+        await DialogService.Show(new DialogOption()
+        {
+            Title = "First Dialog - Large Content",
+            IsScrolling = true,
+            Size = Size.Large,
+            BodyTemplate = builder =>
+            {
+                builder.OpenElement(0, "div");
+                builder.AddAttribute(1, "style", "height: 800px; padding: 20px;");
+                
+                builder.OpenElement(2, "h4");
+                builder.AddContent(3, "First Dialog Content");
+                builder.CloseElement();
+                
+                builder.OpenElement(4, "p");
+                builder.AddContent(5, "This is the first dialog with scrollable content. You should be able to scroll this content.");
+                builder.CloseElement();
+
+                // Add lots of content to make it scrollable
+                for (int i = 1; i <= 50; i++)
+                {
+                    builder.OpenElement(6 + i * 3, "p");
+                    builder.AddContent(7 + i * 3, $"Line {i}: Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.");
+                    builder.CloseElement();
+                }
+                
+                // Add button to open second dialog
+                builder.OpenElement(200, "div");
+                builder.AddAttribute(201, "class", "mt-4");
+                builder.OpenComponent<Button>(202);
+                builder.AddAttribute(203, "Text", "Open Second Dialog");
+                builder.AddAttribute(204, "OnClick", Microsoft.AspNetCore.Components.EventCallback.Factory.Create(this, OpenSecondDialog));
+                builder.CloseComponent();
+                builder.CloseElement();
+                
+                builder.CloseElement();
+            }
+        });
+    }
+
+    private async Task OpenSecondDialog()
+    {
+        await DialogService.Show(new DialogOption()
+        {
+            Title = "Second Dialog - Also Scrollable",
+            IsScrolling = true,
+            Size = Size.Large,
+            BodyTemplate = builder =>
+            {
+                builder.OpenElement(0, "div");
+                builder.AddAttribute(1, "style", "height: 600px; padding: 20px;");
+                
+                builder.OpenElement(2, "h4");
+                builder.AddContent(3, "Second Dialog Content");
+                builder.CloseElement();
+                
+                builder.OpenElement(4, "p");
+                builder.AddAttribute(5, "style", "color: red; font-weight: bold;");
+                builder.AddContent(6, "THIS IS THE SECOND DIALOG - IT SHOULD BE SCROLLABLE TOO!");
+                builder.CloseElement();
+
+                // Add lots of content to make it scrollable
+                for (int i = 1; i <= 30; i++)
+                {
+                    builder.OpenElement(7 + i * 3, "p");
+                    builder.AddContent(8 + i * 3, $"Second Dialog Line {i}: This content should be scrollable in the second dialog. The bug was that this second dialog couldn't scroll.");
+                    builder.CloseElement();
+                }
+                
+                // Add button to open third dialog
+                builder.OpenElement(200, "div");
+                builder.AddAttribute(201, "class", "mt-4");
+                builder.OpenComponent<Button>(202);
+                builder.AddAttribute(203, "Text", "Open Third Dialog");
+                builder.AddAttribute(204, "OnClick", Microsoft.AspNetCore.Components.EventCallback.Factory.Create(this, OpenThirdDialog));
+                builder.CloseComponent();
+                builder.CloseElement();
+                
+                builder.CloseElement();
+            }
+        });
+    }
+
+    private async Task OpenThirdDialog()
+    {
+        await DialogService.Show(new DialogOption()
+        {
+            Title = "Third Dialog - Testing Deep Nesting",
+            IsScrolling = true,
+            Size = Size.Medium,
+            BodyTemplate = builder =>
+            {
+                builder.OpenElement(0, "div");
+                builder.AddAttribute(1, "style", "height: 400px; padding: 20px;");
+                
+                builder.OpenElement(2, "h4");
+                builder.AddContent(3, "Third Dialog Content");
+                builder.CloseElement();
+                
+                builder.OpenElement(4, "p");
+                builder.AddAttribute(5, "style", "color: green; font-weight: bold;");
+                builder.AddContent(6, "THIRD LEVEL DIALOG - SHOULD ALSO SCROLL!");
+                builder.CloseElement();
+
+                // Add content to make it scrollable
+                for (int i = 1; i <= 20; i++)
+                {
+                    builder.OpenElement(7 + i * 2, "p");
+                    builder.AddContent(8 + i * 2, $"Third Dialog Line {i}: Deep nested scrolling test.");
+                    builder.CloseElement();
+                }
+                
+                builder.CloseElement();
+            }
+        });
+    }
+}
\ No newline at end of file
diff --git a/src/BootstrapBlazor/Components/Modal/Modal.razor.scss b/src/BootstrapBlazor/Components/Modal/Modal.razor.scss
index 1e3ff7866ed..406884e19c5 100644
--- a/src/BootstrapBlazor/Components/Modal/Modal.razor.scss
+++ b/src/BootstrapBlazor/Components/Modal/Modal.razor.scss
@@ -85,6 +85,10 @@
 .modal-multiple .modal-dialog {
     position: fixed;
     inset: 0;
+    display: flex;
+    align-items: center;
+    justify-content: center;
+    padding: var(--bs-modal-margin, 0.5rem);
 
     &:last-child:before {
         content: "";
@@ -94,6 +98,19 @@
         opacity: 0.3;
         pointer-events: auto;
     }
+
+    // Ensure scrolling works for multiple dialogs by fixing the modal-content sizing
+    .modal-content {
+        max-height: calc(100vh - var(--bs-modal-margin, 0.5rem) * 2);
+        overflow: hidden;
+        display: flex;
+        flex-direction: column;
+    }
+
+    .modal-body {
+        overflow-y: auto;
+        flex: 1 1 auto;
+    }
 }
 
 .modal-multiple ~ .modal-backdrop {

From 6459ce32545b4e66c979a7561e5f8e2e7f785a55 Mon Sep 17 00:00:00 2001
From: Argo Zhang <argo@live.ca>
Date: Sat, 27 Sep 2025 14:04:29 +0800
Subject: [PATCH 3/5] =?UTF-8?q?fix:=20=E5=A4=9A=E7=BA=A7=E5=BC=B9=E7=AA=97?=
 =?UTF-8?q?=E7=AA=97=E4=BD=93=E5=86=85=E6=BB=9A=E5=8A=A8?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/BootstrapBlazor/Components/Dialog/Dialog.razor.cs | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/BootstrapBlazor/Components/Dialog/Dialog.razor.cs b/src/BootstrapBlazor/Components/Dialog/Dialog.razor.cs
index 679fd8a1b22..5b3f3969e53 100644
--- a/src/BootstrapBlazor/Components/Dialog/Dialog.razor.cs
+++ b/src/BootstrapBlazor/Components/Dialog/Dialog.razor.cs
@@ -166,6 +166,11 @@ private async Task Show(DialogOption option)
 
     private static RenderFragment RenderDialog(int index, Dictionary<string, object> parameter) => builder =>
     {
+        if (index > 0)
+        {
+            parameter[nameof(ModalDialog.IsScrolling)] = true;
+        }
+
         builder.OpenComponent<ModalDialog>(100 + index);
         builder.AddMultipleAttributes(101 + index, parameter);
         builder.SetKey(parameter);

From d95cece2b7c93059a23ea8fff34a00c87cd0f4c6 Mon Sep 17 00:00:00 2001
From: Argo Zhang <argo@live.ca>
Date: Sat, 27 Sep 2025 14:09:20 +0800
Subject: [PATCH 4/5] Revert "Implemented fix for multiple dialog scrolling
 issue"

This reverts commit 42c41361d01c12745d41157d84028a336c87d3e6.
---
 .../Components/Pages/TestScrollDialog.razor   | 134 ------------------
 .../Components/Modal/Modal.razor.scss         |  17 ---
 2 files changed, 151 deletions(-)
 delete mode 100644 src/BootstrapBlazor.Server/Components/Pages/TestScrollDialog.razor

diff --git a/src/BootstrapBlazor.Server/Components/Pages/TestScrollDialog.razor b/src/BootstrapBlazor.Server/Components/Pages/TestScrollDialog.razor
deleted file mode 100644
index 04635390d86..00000000000
--- a/src/BootstrapBlazor.Server/Components/Pages/TestScrollDialog.razor
+++ /dev/null
@@ -1,134 +0,0 @@
-@page "/test-scroll-dialog"
-@using BootstrapBlazor.Server.Components.Components
-@inject DialogService DialogService
-
-<PageTitle>Test Scroll Dialog</PageTitle>
-
-<h1>Test Multiple Dialog Scrolling</h1>
-
-<p>This page tests the multiple dialog scrolling functionality.</p>
-
-<div class="mb-3">
-    <Button Text="Open First Dialog" OnClick="@OpenFirstDialog" />
-</div>
-
-@code {
-    private async Task OpenFirstDialog()
-    {
-        await DialogService.Show(new DialogOption()
-        {
-            Title = "First Dialog - Large Content",
-            IsScrolling = true,
-            Size = Size.Large,
-            BodyTemplate = builder =>
-            {
-                builder.OpenElement(0, "div");
-                builder.AddAttribute(1, "style", "height: 800px; padding: 20px;");
-                
-                builder.OpenElement(2, "h4");
-                builder.AddContent(3, "First Dialog Content");
-                builder.CloseElement();
-                
-                builder.OpenElement(4, "p");
-                builder.AddContent(5, "This is the first dialog with scrollable content. You should be able to scroll this content.");
-                builder.CloseElement();
-
-                // Add lots of content to make it scrollable
-                for (int i = 1; i <= 50; i++)
-                {
-                    builder.OpenElement(6 + i * 3, "p");
-                    builder.AddContent(7 + i * 3, $"Line {i}: Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.");
-                    builder.CloseElement();
-                }
-                
-                // Add button to open second dialog
-                builder.OpenElement(200, "div");
-                builder.AddAttribute(201, "class", "mt-4");
-                builder.OpenComponent<Button>(202);
-                builder.AddAttribute(203, "Text", "Open Second Dialog");
-                builder.AddAttribute(204, "OnClick", Microsoft.AspNetCore.Components.EventCallback.Factory.Create(this, OpenSecondDialog));
-                builder.CloseComponent();
-                builder.CloseElement();
-                
-                builder.CloseElement();
-            }
-        });
-    }
-
-    private async Task OpenSecondDialog()
-    {
-        await DialogService.Show(new DialogOption()
-        {
-            Title = "Second Dialog - Also Scrollable",
-            IsScrolling = true,
-            Size = Size.Large,
-            BodyTemplate = builder =>
-            {
-                builder.OpenElement(0, "div");
-                builder.AddAttribute(1, "style", "height: 600px; padding: 20px;");
-                
-                builder.OpenElement(2, "h4");
-                builder.AddContent(3, "Second Dialog Content");
-                builder.CloseElement();
-                
-                builder.OpenElement(4, "p");
-                builder.AddAttribute(5, "style", "color: red; font-weight: bold;");
-                builder.AddContent(6, "THIS IS THE SECOND DIALOG - IT SHOULD BE SCROLLABLE TOO!");
-                builder.CloseElement();
-
-                // Add lots of content to make it scrollable
-                for (int i = 1; i <= 30; i++)
-                {
-                    builder.OpenElement(7 + i * 3, "p");
-                    builder.AddContent(8 + i * 3, $"Second Dialog Line {i}: This content should be scrollable in the second dialog. The bug was that this second dialog couldn't scroll.");
-                    builder.CloseElement();
-                }
-                
-                // Add button to open third dialog
-                builder.OpenElement(200, "div");
-                builder.AddAttribute(201, "class", "mt-4");
-                builder.OpenComponent<Button>(202);
-                builder.AddAttribute(203, "Text", "Open Third Dialog");
-                builder.AddAttribute(204, "OnClick", Microsoft.AspNetCore.Components.EventCallback.Factory.Create(this, OpenThirdDialog));
-                builder.CloseComponent();
-                builder.CloseElement();
-                
-                builder.CloseElement();
-            }
-        });
-    }
-
-    private async Task OpenThirdDialog()
-    {
-        await DialogService.Show(new DialogOption()
-        {
-            Title = "Third Dialog - Testing Deep Nesting",
-            IsScrolling = true,
-            Size = Size.Medium,
-            BodyTemplate = builder =>
-            {
-                builder.OpenElement(0, "div");
-                builder.AddAttribute(1, "style", "height: 400px; padding: 20px;");
-                
-                builder.OpenElement(2, "h4");
-                builder.AddContent(3, "Third Dialog Content");
-                builder.CloseElement();
-                
-                builder.OpenElement(4, "p");
-                builder.AddAttribute(5, "style", "color: green; font-weight: bold;");
-                builder.AddContent(6, "THIRD LEVEL DIALOG - SHOULD ALSO SCROLL!");
-                builder.CloseElement();
-
-                // Add content to make it scrollable
-                for (int i = 1; i <= 20; i++)
-                {
-                    builder.OpenElement(7 + i * 2, "p");
-                    builder.AddContent(8 + i * 2, $"Third Dialog Line {i}: Deep nested scrolling test.");
-                    builder.CloseElement();
-                }
-                
-                builder.CloseElement();
-            }
-        });
-    }
-}
\ No newline at end of file
diff --git a/src/BootstrapBlazor/Components/Modal/Modal.razor.scss b/src/BootstrapBlazor/Components/Modal/Modal.razor.scss
index 406884e19c5..1e3ff7866ed 100644
--- a/src/BootstrapBlazor/Components/Modal/Modal.razor.scss
+++ b/src/BootstrapBlazor/Components/Modal/Modal.razor.scss
@@ -85,10 +85,6 @@
 .modal-multiple .modal-dialog {
     position: fixed;
     inset: 0;
-    display: flex;
-    align-items: center;
-    justify-content: center;
-    padding: var(--bs-modal-margin, 0.5rem);
 
     &:last-child:before {
         content: "";
@@ -98,19 +94,6 @@
         opacity: 0.3;
         pointer-events: auto;
     }
-
-    // Ensure scrolling works for multiple dialogs by fixing the modal-content sizing
-    .modal-content {
-        max-height: calc(100vh - var(--bs-modal-margin, 0.5rem) * 2);
-        overflow: hidden;
-        display: flex;
-        flex-direction: column;
-    }
-
-    .modal-body {
-        overflow-y: auto;
-        flex: 1 1 auto;
-    }
 }
 
 .modal-multiple ~ .modal-backdrop {

From 865719bde498d3b7fcc7791c664a3b7c6c058f2d Mon Sep 17 00:00:00 2001
From: Argo Zhang <argo@live.ca>
Date: Sat, 27 Sep 2025 14:10:39 +0800
Subject: [PATCH 5/5] chore: bump version 9.10.3

---
 src/BootstrapBlazor/BootstrapBlazor.csproj | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/BootstrapBlazor/BootstrapBlazor.csproj b/src/BootstrapBlazor/BootstrapBlazor.csproj
index 917abf50715..36199d92bab 100644
--- a/src/BootstrapBlazor/BootstrapBlazor.csproj
+++ b/src/BootstrapBlazor/BootstrapBlazor.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk.Razor">
 
   <PropertyGroup>
-    <Version>9.10.2</Version>
+    <Version>9.10.3</Version>
   </PropertyGroup>
 
   <ItemGroup>