Changeset 3673748

Timestamp:

08/31/2026 09:21:57 AM (8 weeks ago)

Author:

megamenu

Message:

Fix: Theme editor - saving could submit stale/empty content for CodeMirror-backed fields (e.g. Pro's toggle bar HTML block) if saved before the field's own change handler had run

Location:

megamenu/trunk

Files:

• js/admin/theme-editor.js (modified) (1 diff)
• readme.txt (modified) (1 diff)

megamenu/trunk/js/admin/theme-editor.js

@@ -735 +735 @@
         $(".theme_result_message").remove();
 
+        // Flush every CodeMirror instance in the form (custom CSS, and any
+        // companion-plugin textareas such as Pro's toggle bar HTML block) back
+        // into its textarea before serializing — CodeMirror doesn't keep the
+        // underlying textarea's value in sync on every keystroke, so without
+        // this a save triggered before an individual field's own blur/change
+        // handler runs submits stale (often empty) content for that field.
+        $form.find("textarea").each(function () {
+            window.megamenuCodeEditor.save(this);
+        });
+
         $.ajax({
             url:  ajaxurl,

megamenu/trunk/readme.txt

@@ -138 +138 @@
 * Fix: WPML compatibility - menu item depth detection could fail on PHP 8 when a synced translation's parent ID was missing, breaking Mega Menu grid layouts for affected items
 * Fix: Menu item modal - lower z-index so third-party builders opened from within it (e.g. SiteOrigin Page Builder's "Layout Builder" widget) render above it instead of hidden underneath
+* Fix: Theme editor - saving could submit stale/empty content for CodeMirror-backed fields (e.g. Pro's toggle bar HTML block) if saved before the field's own change handler had run
 
 = 3.10.6 =