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