Skip to content
Browse files

Fixed so remove jQuery method removes the TinyMCE instance as well as…

… the specified textarea.
  • Loading branch information...
1 parent 2558dc2 commit 9736302197fc4cf7452b030951a250dca742f123 @spocke spocke committed Feb 23, 2010
Showing with 5 additions and 4 deletions.
  1. +2 −1 changelog.txt
  2. +3 −3 jscripts/tiny_mce/classes/adapter/jquery/jquery.tinymce.js
View
3 changelog.txt
@@ -1,4 +1,4 @@
-Version 3.3rc1 (2010-xx-xx)
+Version 3.3rc1 (2010-02-23)
Fixed bug with new legacyoutput plugin not working correctly on it's own.
Fixed bug some performance issues with removing text formats.
Fixed bug where TinyMCE specific attributes wasn't removed properly by remove format.
@@ -7,6 +7,7 @@ Version 3.3rc1 (2010-xx-xx)
Fixed bug where the search/replace logic could produce an infinite loop on IE for reverse searches.
Fixed bug where cloning formats in cells didn't work properly on IE.
Fixed bug where IE6 would produce a horizontal scroll bar.
+ Fixed so remove jQuery method removes the TinyMCE instance as well as the specified textarea.
Fixed so selected rows and cells gets updated using the row/cell properties dialogs.
Version 3.3b2 (2010-02-04)
Fixed bug where sometimes img elements would be removed by split method in DOMUtils.
View
6 jscripts/tiny_mce/classes/adapter/jquery/jquery.tinymce.js
@@ -172,9 +172,9 @@
// automatically destroyed by the TinyMCE API
function applyPatch() {
// Removes any child editor instances by looking for editor wrapper elements
- function removeEditors(remove) {
+ function removeEditors(name) {
// If the function is remove
- if (remove) {
+ if (name === "remove") {
this.each(function(i, node) {
var ed = tinyMCEInstance(node);
@@ -293,7 +293,7 @@
var origFn = jQueryFn[name] = $.fn[name];
$.fn[name] = function() {
- removeEditors.apply(this, name === "remove");
+ removeEditors.call(this, name);
return origFn.apply(this, arguments);
};

0 comments on commit 9736302

Please sign in to comment.
Something went wrong with that request. Please try again.