Closed Bug 480247 Opened 17 years ago Closed 16 years ago

Undo stack is broken which removes all content from an opened test

Categories

(Testing Graveyard :: Mozmill, defect, P4)

Product:
Testing Graveyard
Component:
Mozmill
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: whimboo, Assigned: adam.christian)

Details

(Whiteboard: [verified-mozmill-1.2])

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b3pre) Gecko/20090224 Shiretoko/3.1b3pre Ubiquity/0.1.5 ID:20090224022145 and latest trunk 1. Open a test file 2. Hit Cmd/Ctrl+Z Undo should only be possible for changes happened in the current session.
The way open works is that it creates a pointer to the file in JS, then creates a new tab and fills it with the files contents, so when it first opens for a split second its empty, and then the content shows up -- so ctrl+z undoes the last action which is filling the contents with the file. I see what you mean by ctl z probably shouldn't do anything at that point, but how many people are going to open a file and immediately hit ctrl+z, so I think we are going to have to punt this one to future.
It's the same as when you have edited the file and hold down Cmd+Z to revert all those changes. Is there a way to tell the editor that the undo stack should be cleared? Is it probably an error in the underlying editor component?
Whiteboard: [mozmill-1.2]
Priority: -- → P4
Opening MozMill and pressing Cmd/Ctrl+Z adds a line to the blank file and extends the file tab and then "undos" that undo.
It's more obvious when having a test already open. Create a new test in another tab and hit Cmd/Ctrl+Z there.
Committed revision 451.
Assignee: nobody → adam.christian
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Verified fixed with r451.
Status: RESOLVED → VERIFIED
Whiteboard: [mozmill-1.2] → [verified-mozmill-1.2]
In 1.4.2b3 this issue is back in the IDE STR: 1. Open a file 2. Make a change. 3. Hold down Cmd/Ctrl+Z ER: Undo's the changes since opening the file (single correction) AR: The file is wiped out I am reopening this as I am seeing this bug again
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Keywords: regression
Please file a new bug for Bespin. This bug has been fixed for Mozmill 1.2.
Status: REOPENED → RESOLVED
Closed: 17 years ago → 16 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Keywords: regression
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.