Ticket #3105 (closed defect: fixed)

Opened 21 months ago

Last modified 11 months ago

Paint: undo does not work reliably

Reported by: erikos Owned by: humitos
Priority: Unspecified by Maintainer Milestone: 0.96
Component: Paint Version: 0.93.x
Severity: Unspecified Keywords: 12.1.0, patch, olpc-test-passed
Cc: humitos Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

I have been using today Paint for a while and 'undo' has not been working for me reliably. Sometimes more than one step did get undone. For example I did color different areas (with the bucket tool) and then did press 'undo' and it jumped back too an older version and did not just undo the last step.

Paint version 36

Attachments

0001-Undo-fixed-SL-3105.patch Download (1.2 KB) - added by humitos 13 months ago.
spin-undo-list-test.patch Download (5.1 KB) - added by humitos 13 months ago.
0001-Undo-after-using-bucket-tool-fixed-SL-3105.patch Download (1.8 KB) - added by humitos 13 months ago.
This patch solves this issue

Change History

Changed 13 months ago by humitos

  • cc humitos added

Yes, I can confirm this. I've just tested it on XO 1.75 os12 and Undo is not working properly.

In fact, if you

  1. color different areas as erikos said
  2. press the Undo button
  3. color some new areas

The ability to Undo again is lost. You just can click on Redo and it does "something strange".

Changed 13 months ago by humitos

  • owner changed from godiard to humitos
  • status changed from new to accepted

Changed 13 months ago by humitos

Changed 13 months ago by humitos

  • keywords patch added

This patch solves this issue, but there is a case that I couldn't find yet:

  1. Draw anything in 5 steps. For example, five lines
  2. Click twice on "Undo"
  3. Click once on "Redo"

The "Redo" action will not work. But if you press this button again you will see the "Redo" action done. I'm pretty sure that it's related with the index (self._undo_index) of the self._undo_list but I couldn't fix it yet.

Changed 13 months ago by humitos

Changed 13 months ago by humitos

I added a patch that adds a new Spin Button to show every "undo item" in the self._undo_list. This is useful to test if the "undo item" is being shown or not.

I don't know why but every time that I use the bucket twice and click on "Undo" both of the fills are undone. After that when I press on "Redo" just the first one is redone.

Changed 13 months ago by humitos

This patch solves this issue

Changed 13 months ago by manuq

  • keywords 12.1.0, patch, olpc-test-pending added; patch removed
  • status changed from accepted to closed
  • resolution set to fixed
  • milestone changed from Unspecified by Release Team to 0.96

Thanks! Pushed.

Changed 11 months ago by greenfeld

  • keywords olpc-test-passed added; olpc-test-pending removed

Undo and redo support seems to work properly in Paint in 12.1.0 os18.

Note: See TracTickets for help on using tickets.