This issue tracker is closed. Please visit UPPAAL issue tracker at Github instead.

Bug 403 - Undo has unexpected behaviour
Summary: Undo has unexpected behaviour
Status: ASSIGNED
Alias: None
Product: UPPAAL
Classification: Unclassified
Component: GUI (show other bugs)
Version: 4.0.5
Hardware: All All
: P2 enhancement
Assignee: Gerd Behrmann
URL:
Depends on:
Blocks:
 
Reported: 2007-03-12 11:00 CET by Lars T. Mikkelsen
Modified: 2007-05-15 23:23 CEST (History)
0 users

See Also:
Architecture:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lars T. Mikkelsen 2007-03-12 11:00:04 CET
The Undo function of the text editor has some very unexpected behaviour - and in some cases quite critical behaviour. It's a bit difficult to describe the expected behaviour exactly, but I have two examples. Enter "abc", delete the "b", Undo - I would expect the "b" to reappear, but instead the line is removed. Another, and more critical, example: make a change on one line, make a change on another line, Undo - I would expect only the last change to be undone, but instead both changes are undone - this way it's very likely to undo a change without noticing.
Comment 1 Gerd Behrmann 2007-03-12 11:07:05 CET
This is a known problem.

The issue you are hinting at is the granularity of a change. You mention that in your last case, both changes are undone. In reality, only the last change is undone, however the last change covers both lines. If you continue experimenting, you will notice, that all changes in a text-box until that text-box looses focus, are considered to be one change. The undo mechanism was really designed to work on the automaton level, and the behavior is really inappropriate for the declaration sections.

The planned fix for this was to use a timer to commit changes in a declaration second every second or so. Maybe it is time to finally make this change :-)
Comment 2 Gerd Behrmann 2007-05-15 23:23:59 CEST
At least a partial fix has been committed to the 4.0 branch in rev. 3065. The text in a text area is now committed to the underlying model representation after at most one second. This will limit how much is undone at a time.