Ticket #2708 (new enhancement)

Opened 2 years ago

Last modified 4 months ago

Paint should save control settings in its Journal entry

Reported by: greenfeld Owned by: godiard
Priority: Unspecified by Maintainer Milestone: 1.0
Component: Paint Version:
Severity: Unspecified Keywords:
Cc: humitos Distribution/OS: OLPC
Bug Status: Unconfirmed

Description

I started work on a pretty painting with different colors and custom sizes for shapes in Paint-31. I then saved it to a Journal entry, and reopened it later to continue working.

The picture was there, but all my custom color, pixel, and fill settings were lost.

If possible we should save these. If someone screws these up too badly we should suggest/support importing the picture from the old instance of Paint into a new instance of Paint without resetting the controls.

Attachments

save-restore-brush-properties.patch Download (16.0 KB) - added by humitos 4 months ago.
A very old patch that I was working on 10 months ago. Maybe it is useful to take some ideas or chunks of code

Change History

  Changed 14 months ago by humitos

I'm going to start working on this ticket.

  Changed 14 months ago by humitos

I could save the state using write_file and load it using read_file without any problem. I'm restoring the last tool used by the user but I can't restore the colors: "fill color" and "stroke color" yet.

I'm using simplejson to store the data as string into self.metadata['state'].

I didn't find the way to let the GUI know that should use another color instead the default one. With "GUI" I refer to Area class.

  Changed 9 months ago by godiard

  • keywords 11.3.0 added

  Changed 6 months ago by godiard

  • milestone changed from Unspecified by Release Team to 1.0

follow-ups: ↓ 6 ↓ 7   Changed 4 months ago by manuq

  • cc humitos added

Humitos, do you still have your patch around? Gonzalo, any chance we can see at least the part that works landing?

in reply to: ↑ 5   Changed 4 months ago by godiard

  • keywords 11.3.0 removed
  • version 0.92.x deleted

Replying to manuq:

Gonzalo, any chance we can see at least the part that works landing?

Yes :)

Changed 4 months ago by humitos

A very old patch that I was working on 10 months ago. Maybe it is useful to take some ideas or chunks of code

in reply to: ↑ 5   Changed 4 months ago by humitos

Replying to manuq:

Humitos, do you still have your patch around?

I've just attached the patch to this ticket. I'm not sure if it will be useful, but maybe we can take some chunks of code or some ideas.

Note: See TracTickets for help on using tickets.