Merge pull request #41 from zyhfish/bug/DNN-8065

DNN-8065: sync title attribute with alt value when save image.
DNN-Connect · Dec 4, 2015 · d032df8 · d032df8
2 parents 57111a2 + 54c031f
commit d032df8
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 0 deletions.
diff --git a/DNNConnect.CKEditorProvider.csproj b/DNNConnect.CKEditorProvider.csproj
@@ -227,6 +227,7 @@
     <Content Include="css\jquery.notification.css" />
     <Content Include="css\Options.css" />
     <Content Include="Install\DNNCkeditor.license.txt" />
+    <Content Include="js\editorOverride.js" />
     <Content Include="js\jquery.ckeditor.adapter.js" />
     <Content Include="js\jquery.notification.js" />
     <Content Include="js\jquery.ui.controls.js" />

diff --git a/Web/EditorControl.cs b/Web/EditorControl.cs
@@ -1287,6 +1287,8 @@ private void RegisterCKEditorLibrary()
                     Globals.ResolveUrl("~/Providers/HtmlEditorProviders/DNNConnect.CKE/ckfinder/ckfinder.js"));
             }
 
+            ClientResourceManager.RegisterScript(Page, Globals.ResolveUrl("~/Providers/HtmlEditorProviders/DNNConnect.CKE/js/editorOverride.js"));
+
             // Load Custom JS File
             if (!string.IsNullOrEmpty(currentSettings.CustomJsFile)
                 && !cs.IsClientScriptIncludeRegistered(csType, "CKCustomJSFile"))

diff --git a/js/editorOverride.js b/js/editorOverride.js
@@ -0,0 +1,19 @@
+(function ($) {
+    if (CKEDITOR) {
+        CKEDITOR.on('dialogDefinition', function(evt) {
+            var dialog = evt.data;
+            if (dialog.name === 'image') {
+                // Get dialog definition.
+                var def = evt.data.definition;
+                var originOkFunc = def.onOk;
+
+                def.onOk = function () {
+                    originOkFunc.apply(this, arguments);
+                    var $element = $(this.imageElement);
+                    $element.attr('title', $element.attr('alt'));
+                };
+
+            }
+        });
+    }
+})(jQuery);