Re: Support for Undo

[Alexander Larsson <alexl redhat com>]

On Wed, 2011-11-02 at 15:51 +0000, Philip Withnall wrote:
> On Wed, 2011-11-02 at 10:20 -0500, Erick Pérez Castellanos wrote:
> > Hi:
> > I've been looking to this
> > bug:https://bugzilla.gnome.org/show_bug.cgi?id=657975 and after seeing
> > the discussion with the folks devs, I think the final choice is to wait,
> > if the user doesn't press Undo, then commit to libfoklks the operation
> > right ?.
> 
> That's correct.

Yeah, and add some magic in the Contacts store to hide the persona.

> > Well, in that case, I say, why not use the same pattern for linking.
> 
> Because linking is, by design, always undoable. (Compared to merging,
> which isn't.) There should be no need to delay committing a linking
> operation, since undoing it should be as simple as just calling the
> unlinking code.

Also, we must do the linking operation so that folks can link the
personas together so that we can update the UI to the new state. Without
that we'd have to second-guess what folks would do in order to present
the new state of the UI.