[dw-free] RTE + user tags in Safari, Opera, Chrome has issues. Text after the user tag gets put into

[fu]

[commit: http://hg.dwscoalition.org/dw-free/rev/516346bc07fe]

http://bugs.dwscoalition.org/show_bug.cgi?id=2542

Fix issue with RTE + user tags, by adding a space after the user tag in the
same step as you add the user tag (otherwise, the RTE HTML parser gets
confused, and insists on still putting the space/any further text inside one
of the outer layers of the user tag)

Patch by LiveJournal, packaged for Dreamwidth by fu.

Files modified:

• htdocs/stc/fck/editor/plugins/livejournal/fckplugin.js

--------------------------------------------------------------------------------
diff -r efdabfbecf0c -r 516346bc07fe htdocs/stc/fck/editor/plugins/livejournal/fckplugin.js
--- a/htdocs/stc/fck/editor/plugins/livejournal/fckplugin.js	Wed Apr 21 21:41:53 2010 -0500
+++ b/htdocs/stc/fck/editor/plugins/livejournal/fckplugin.js	Thu Apr 22 03:17:04 2010 -0700
@@ -63,15 +63,13 @@ LJUserCommand.Execute=function() {
             data.ljuser = data.ljuser.replace(/<\/span>/,'</div>');
             UserTagCache[username + "_" + site] = data.ljuser;
 
-            FCK.InsertHtml(data.ljuser);
-            FCK.InsertHtml('&nbsp;')
+            FCK.InsertHtml(data.ljuser + "&nbsp;");
             if (selection != '') FCKSelection.Collapse();
             FCK.Focus();
         }
     
         if ( UserTagCache[postData.username+"_"+postData.site] ) {
-            FCK.InsertHtml(UserTagCache[postData.username+"_"+postData.site]);
-            FCK.InsertHtml('&nbsp;')
+            FCK.InsertHtml(UserTagCache[postData.username+"_"+postData.site] + "&nbsp;");
             if (selection != '') FCKSelection.Collapse();
             FCK.Focus();
         } else {
--------------------------------------------------------------------------------

Comments

[yvi]

And this, ladies and gentlemen, is why I wouldn't touch the RTE code with... anything.

[fu]

Oh man, I know what you mean. I feel all dirty and confused whenever I wander through the RTE code ahhahaha.