On Wed, 2012-03-28 at 10:28 +0100, Dave Page wrote:
> That looks OK to me - thanks. Guillaume, what do you think (it's
> fixing a bug in your code :-) ). Basically the problem is in the code
> you added to prevent the same properties dialogue being opened twice,
> or dropping of objects that are open in dialogues. The issue was that
> the code always assumed the selected item in the treeview was the item
> on which it was acting, which is not always the case (for example, you
> might double-click an item in the listview).
>
It seems good to me. I don't reproduce the bug, so I can't say if this
specific patch fixed it. But the patch seems good.
--
Guillaume
http://blog.guillaume.lelarge.info
http://www.dalibo.com