Undo and Redo are completely broken in full page mode

[anonymous]

Nothing happens when pressing undo if fullpage is used.

[PoeuffF]

I've tried with IE and mozilla but it works. All the operations works but you can't undo the fullpage's operation.

I think that's it your pb.

[PoeuffF]

I've tried with IE and mozilla but it works. All the operations works but you can't undo the fullpage's operation.

I think that's it your pb?

[Nisse]

First I write something and then deletes it. If I push the undo button nothing happens. It don't work on the example page online either. What do you mean by "can't undo the fullpage's operation"?. This problem concerns the Fullpage plugin aka editing a page with header and all, not the fullscreen version.

[mharrisonline]

This happens in IE and Firefox, either with the full page plugin or with the fullpage configured to true in htmlarea.js so it's not a plugin issue.

[mharrisonline]

I have a version of Xinha where htmlare.js was last changed on 5/19 and it works, on a version that was last changed on 5/25 it doesn't work.

[anonymous]

I tried all the revisions around that time, and it worked perfectly until revision 177, which seems to have broken it:

Revision 177 (by gogo, 05/19/05 10:28:30)  * Minor comment changes.
* Refactor events for form submit, reset and window unload
* Fix reset handler to properly use the original textarea content
* Minor changes to setHTML etc
* Use onload for iframe to determine when to initIframe, instead of a 


timeout 
* Remove overlooked debugging statement in loadPlugins
* Change some event registrations to use addDom0Event rather than 
element.onxxx = something
* Add HTMLArea.flushEvents() as an implementation of #30* - sadly, 
doesn't work. This is not enabled in the full_example, use 
examples/testbed.html to test it out.
* Added methods HTMLArea.addDom0Event and HTMLArea.prependDom0Event
which are to be used instead of element.onxxxxx = something;

[mharrisonline]

Gyahh! I hate this stuff. I figured out how to make undo work in full-page mode again. I went through the diff file for revision 177 of htmlarea.js, and tried replacing all changed code (in the current download of htmlarea.js) with the previous code.

I narrowed it down to the changes made to this function: HTMLArea.prototype.setHTML = function(html) {

Current broken version:

// completely change the HTML inside
HTMLArea.prototype.setHTML = function(html) {
  switch (this._editMode)
  {
    case "wysiwyg"  :
    {
      if (!this.config.fullPage)
      {
        this._doc.body.innerHTML = html;
      }
      else
      {
        this._doc.setFullHTML(html);
      }
    }
    break;

    case "textmode" :
    {
      this._textArea.value = html;
    }
    break;

    default	        :
    {
      alert("Mode <" + mode + "> not defined!");
    }
    break;
  }
  return false;
};

Original working version:

// completely change the HTML inside
HTMLArea.prototype.setHTML = function(html) {
  switch (this._editMode) {
      case "wysiwyg"  :
    if (!this.config.fullPage)
      this._doc.body.innerHTML = html;
    else
      // this._doc.documentElement.innerHTML = html;
      this._doc.body.innerHTML = html;
    break;
      case "textmode" : this._textArea.value = html; break;
      default	    : alert("Mode <" + mode + "> not defined!");
  }
  return false;
};

I hope this doesn't break anything else, but this fixes undo in full page mode.

[gocher]

I see, the Error is in setFullHTML(html);

[gocher]

this._doc.setFullHTML(html) is not a function this.setFullHTML(html) is to set but thats not all!

With a little Testfile it works but with the Example it doesn't works! HTMLArea.RE_body = /<body[^{>]*>((.|\n)*?)<\/body>/i; is good but HTMLArea.RE_body = /<body[}>]*>((.|\n|\r|\t)*?)<\/body>/i; is better!

Checked in Changeset 258!