On Sun, Mar 30, 2003 at 01:35:18AM +0100, Alfredo Braunstein wrote:

> Can you try this? I've put inside #if 0 the code that make local the
> mouseclick position before calling edit or localDispatch on a
> !isHiglyEditable() child inset, and added a simple localdispatch for
> insetbuttons that call edit().
> This works because simple insets (that do not have an overloaded
> localDispatch()) do not use the cursor position received on edit(), AFAIK.

Hmm, OK. Then where is the comment above the #if 0 saying this ? I willl
add it, and test the patch in a bit. Thanks.

> I think that we should switch to _always_ pass the local coordinates to
> insets in the future, but at least it's more uniform with this patch.

Yes, we really need to clean the whole thing altogether. It's  mostly a
residue of when the cursor presses were not lfuns. I'm in favour of
dropping ->edit() stuff altogether. If an inset doesn't care, it can
inherit a default or something

regards
john

Reply via email to