Jean-Marc Lasgouttes wrote:
> Sorry indeed. InsetCommand should maybe have something like
> InsetGraphics:
> void InsetGraphics::edit(Cursor & cur, bool)
> {
>         InsetGraphicsMailer(*this).showDialog(&cur.bv());
> }
>
> (and then should use it when handling MOUSE_RELEASE).

I see. I'll have a look.

> BTW, does the approach I proposed work with InsetGraphics?

Yes.

> > At least I detected an error: InsetIndex returns NOT_EDITABLE for
> > editable(), which is clearly wrong.
>
> No, it returns IS_EDITABLE. InsetPrintIndex returns NOT_EDITABLE.

Indeed. My bad.

Jürgen

Reply via email to