Bug 7160

Summary: TinyMCE: Undo toolbar command doesn't undo the previous operation
Product: WebKit Reporter: Chris Petersen <c.petersen87>
Component: HTML EditingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED WONTFIX    
Severity: Normal CC: ian, markmalone
Priority: P2 Keywords: InRadar
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://tinymce.moxiecode.com/example.php?example=true
Bug Depends on:    
Bug Blocks: 6627    

Chris Petersen
Reported 2006-02-09 11:15:49 PST
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.
Attachments
Chris Petersen
Comment 1 2006-02-09 11:16:49 PST
Note: All of these cases do work correctly in PPC FF 1.5
Chris Petersen
Comment 2 2006-02-09 11:32:39 PST
This bug is also in Radar as <rdar://4439701 >
Justin Garcia
Comment 3 2006-09-19 17:15:48 PDT
This is Moxiecode's bug. Browsing the source I don't see any calls to add an undo level in the Safari code paths.
David Kilzer (:ddkilzer)
Comment 5 2006-09-19 17:32:09 PDT
This bug could be reopened as an evangelism bug.
Note You need to log in before you can comment on or make changes to this bug.