Re: [Planner Dev] Bugs in the undo system: Report to reach 0.12

["lincoln phipps openmutual net" <lincoln phipps openmutual net>]

Alvaro del Castillo wrote:

> Hmmm, what I am doing is when you remove a property, save all data from
> this property (GValues) to a hash table. In the hash table I use the
> MrpTask o MrpResource as the key and the data is the GValue for the
> property. If you undo the property removal then you add the property
> another time and set all the tasks or resources to the GValues they had.
>
> Do you find some prob with this approach? I am very near to finish it
> adn we can test it.


No that sounds perfect ! - Alvaro, you're a better coder than I !.

With, Undo names are inconsistent,
http://bugzilla.gnome.org/show_bug.cgi?id=144742

what do we plan for the actual text strings ?. Wouldn't it
be nice to have some convention so as to make life easier
for the translators ?.

e.g. just make it 3 words,

"Action" "Thing" "Attribute"

e.g.,

"Change Task Name"

where action is one of,

	Change,
	Add,
	Delete,
	Insert

thing is,

	Task,
	Resource,
	Project

atrributes are many.

Hasta Luego,
Lincoln.

> Cheers
>
> -- Alvaro
>
>
> > Then on UNDO (the undo label would say "Restore Property Values")
> > I would undo that and then call the undo again to undo
> > the property (I guess we can do that ?).
> >
> > Well tat was my plan though it was trying to work out how
> > to get the column data that was bugging me and I've
> > stopped there.
> >
> > Rgds,
> > Lincoln.
> >
> > > Cheers
> > >
> > > -- Alvaro
> > >
> > >
> > > ------------------------------------------------------------------------
> > >
> > > _______________________________________________
> > > Planner-dev mailing list
> > > Planner-dev lists imendio com
> > > http://lists.imendio.com/mailman/listinfo/planner-dev
> >
> > _______________________________________________
> > Planner-dev mailing list
> > Planner-dev lists imendio com
> > http://lists.imendio.com/mailman/listinfo/planner-dev
>
>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> Planner-dev mailing list
> Planner-dev lists imendio com
> http://lists.imendio.com/mailman/listinfo/planner-dev