Jean-Marc Lasgouttes wrote:

> I am not sure the patch is really right: if I insert a TOC inset, it
> will mean that the selection is not replaced anymore, but it stays
> here. Actually, only INDEX requires to use the selection without
> deleting it.

You're right, the new behaviour should only apply to index insets (for now;
maybe there will be similar cases eventually). That's why I tried to
implement a new special case to doInsertInset in my first patch. 

> What about this different version?

What about moving your code there and adding another bool for the case "just
copy the selection and paste it inside the inset"? Or do you think this is
overkill?

Jürgen

Reply via email to