TinyMCE: Undo toolbar command doesn't undo the previous operation

[Chris Petersen]

Summary: Clicking the Undo toolbar button DOESN'T undo the last previous operation that was applied in the textarea field. I have several cases were this occurs.


Steps to reproduce:


Undoing a style applied to text

1) With TOT webkit, go to http://tinymce.moxiecode.com/example.php?example=true
2) Delete all elements in the first textarea field;  then type a new sentence in this textarea field
3) Select the entire sentence or a single word
4) Click on B toolbar icon and verify style is applied
5) Click on Undo toolbar icon. Instead of removing the bold style applied, it removes the entire line of text
6) This does work if I choose the Undo from the Edit menu.


Undoing the LAST style applied to text

1) With TOT webkit, go to http://tinymce.moxiecode.com/example.php?example=true
2) Delete all elements in the first textarea field;  then type a new sentence in this textarea field
3) Select the entire sentence or a single word
4) Click on B, I, U toolbar icon and verify all styles are applied
5) Click on Undo toolbar icon. Instead of removing that last style applied, it removes the entire line of text
6) This does work if I choose the Undo from the Edit menu.


Undoing pasted text 

1) With TOT webkit, go to http://tinymce.moxiecode.com/example.php?example=true
2) Delete all elements in the first textarea field;  then type a new sentence in this textarea field
3) Select the entire sentence or a single word
4) Press command -C. Place the caret at the end of the sentence and press return key.
5) Caret appears on new line. Now press command -v to paste the text
6) Click on Undo toolbar icon. Instead of removing the pasted text , it removes the both lines of text
7) This does work if I choose the Undo Paste from the Edit menu.

[Chris Petersen]

Note: All of these cases do work correctly in PPC FF 1.5

[Chris Petersen]

This bug is also in Radar as <rdar://4439701 >

[Justin Garcia]

This is Moxiecode's bug.  Browsing the source I don't see any calls to add an undo level in the Safari code paths.

[Justin Garcia]

Filed http://sourceforge.net/tracker/index.php?func=detail&aid=1561817&group_id=103281&atid=635682

[David Kilzer (:ddkilzer)]

This bug could be reopened as an evangelism bug.