From 4f9294b646c267beb31672ae9aca45ed52f2b987 Mon Sep 17 00:00:00 2001
From: Trevor Forrester <support@forrestitservices.org>
Date: Mon, 20 Jul 2026 21:20:52 +1000
Subject: [PATCH] Fix Blog export dialog styling on DNN 10

The Blog export dialog was being created correctly, but DNN 10 did not reliably apply the classes supplied through the legacy dialogClass option to the generated jQuery UI dialog wrapper.

This caused the Export title, text, download link, and buttons to appear outside a properly styled popup.

This change retrieves the generated dialog widget when it opens and applies the required DNN popup classes directly:

dnnFormPopup
dnnClear
dnnBlogExportPopup

The button-pane selectors were also scoped to the current Export dialog to avoid affecting other dialogs on the page.

Tested successfully on:

DNN 9.13.1
clean DNN 10.3.2 installation
production DNN 10.3.2 installation
Blog export and BlogML download
Blog import
---
 Server/Blog/Manage.ascx | 142 ++++++++++++++++++++++++++++------------
 1 file changed, 100 insertions(+), 42 deletions(-)

diff --git a/Server/Blog/Manage.ascx b/Server/Blog/Manage.ascx
index 563b7228..131b6b61 100644
--- a/Server/Blog/Manage.ascx
+++ b/Server/Blog/Manage.ascx
@@ -87,46 +87,104 @@
 
 <script type="text/javascript">
 (function ($, Sys) {
- $('#tabs').dnnTabs();
- var selectedBlog;
- var $dialogexport = $('<div class="dnnDialog"></div>')
-		.html('<p><%=LocalizeJSString("Export.Help") %></p><p><a id="blogMLDownloadLink" style=""><%=LocalizeJSString("Download") %></a></p>')
-		.dialog({
-		 autoOpen: false,
-		 resizable: false,
-		 dialogClass: 'dnnFormPopup dnnClear',
-		 title: '<%=LocalizeJSString("Export") %>',
-		 height: 250,
-		 width: 500,
-		 open: function (e) {
-		  $('#blogMLDownloadLink').hide();
-		  $('.ui-dialog-buttonpane').find('button:contains("<%=LocalizeJSString("Export") %>")').addClass('dnnPrimaryAction');
-		  $('.ui-dialog-buttonpane').find('button:contains("<%=LocalizeJSString("Cancel") %>")').addClass('dnnSecondaryAction');
-		 },
-		 buttons: [
-    {
-     text: '<%=LocalizeJSString("Cancel") %>',
-     click: function () {
-      $(this).dialog("close");
-     }
-    },
-    {
-     text: '<%=LocalizeJSString("Export") %>',
-     click: function () {
-      $('.ui-dialog-buttonpane').find('button:contains("<%=LocalizeJSString("Export") %>")').attr("disabled", "disabled");
-      blogService.exportBlog(selectedBlog, function (returnValue) {
-       $('.ui-dialog-buttonpane').find('button:contains("<%=LocalizeJSString("Export") %>")').removeAttr("disabled");
-       $('#blogMLDownloadLink').attr('href', returnValue.Result);
-       $('#blogMLDownloadLink').show();
-      });
-     }
-    }
-    ]
-		});
- $('.exportlink').click(function () {
-  selectedBlog = $(this).parent().attr('data-blogid');
-  $dialogexport.dialog('open');
-  return false;
- });
-} (jQuery, window.Sys));
+    $('#tabs').dnnTabs();
+
+    var selectedBlog;
+
+    var $dialogexport = $('<div class="dnnDialog"></div>')
+        .html(
+            '<p><%=LocalizeJSString("Export.Help") %></p>' +
+            '<p><a id="blogMLDownloadLink" style="">' +
+            '<%=LocalizeJSString("Download") %>' +
+            '</a></p>'
+        )
+        .dialog({
+            autoOpen: false,
+            resizable: false,
+
+            /*
+             * Retain the original option for older DNN/jQuery UI versions.
+             * The open event below also applies these classes directly to
+             * the generated dialog wrapper for DNN 10 compatibility.
+             */
+            dialogClass: 'dnnFormPopup dnnClear dnnBlogExportPopup',
+
+            title: '<%=LocalizeJSString("Export") %>',
+            height: 250,
+            width: 500,
+
+            open: function () {
+                var $dialogWidget = $(this).dialog('widget');
+                var $buttonPane = $dialogWidget.find('.ui-dialog-buttonpane');
+
+                /*
+                 * DNN 10 does not reliably apply dialogClass to the generated
+                 * outer wrapper. Adding the classes here restores the normal
+                 * DNN popup appearance.
+                 */
+                $dialogWidget.addClass(
+                    'dnnFormPopup dnnClear dnnBlogExportPopup'
+                );
+
+                $('#blogMLDownloadLink').hide();
+
+                $buttonPane
+                    .find(
+                        'button:contains("<%=LocalizeJSString("Export") %>")'
+                    )
+                    .addClass('dnnPrimaryAction');
+
+                $buttonPane
+                    .find(
+                        'button:contains("<%=LocalizeJSString("Cancel") %>")'
+                    )
+                    .addClass('dnnSecondaryAction');
+            },
+
+            buttons: [
+                {
+                    text: '<%=LocalizeJSString("Cancel") %>',
+
+                    click: function () {
+                        $(this).dialog('close');
+                    }
+                },
+                {
+                    text: '<%=LocalizeJSString("Export") %>',
+
+                    click: function () {
+                        var $dialogWidget = $(this).dialog('widget');
+
+                        var $exportButton = $dialogWidget
+                            .find('.ui-dialog-buttonpane')
+                            .find(
+                                'button:contains("<%=LocalizeJSString("Export") %>")'
+                            );
+
+                        $exportButton.attr('disabled', 'disabled');
+
+                        blogService.exportBlog(
+                            selectedBlog,
+
+                            function (returnValue) {
+                                $exportButton.removeAttr('disabled');
+
+                                $('#blogMLDownloadLink')
+                                    .attr('href', returnValue.Result)
+                                    .show();
+                            }
+                        );
+                    }
+                }
+            ]
+        });
+
+    $('.exportlink').click(function () {
+        selectedBlog = $(this).parent().attr('data-blogid');
+        $dialogexport.dialog('open');
+
+        return false;
+    });
+
+}(jQuery, window.Sys));
 </script>
\ No newline at end of file