On Thu, Jan 03, 2002 at 04:36:37PM +0100, Juergen Vigna wrote:
> > But now that Formula pretents being a text inset ( isTextInset() -> true )
> > we might as well fake replaceSelectionWithString() and
> > setSelectionOverString, too.
> [snip]
> >   - use bv->getSearchableInset()->replaceSelectionWithString(...) instead of
> >     bv->getLyXText()->replaceSelectionWithString(...) in lyxfind.C
> 
> Well in all this you forgot that this was made for the REAL text too and the
> real text is NOT an inset.

I think I have not forgotten this, I rather never knew this...

> So the above does NOT work. What would work is to
> use something like this
> 
> if (bv->theLockingInset()) {
>    ...
>    bv->theLockingInset()->replaceSelectionWithString(...);
>    ...
> } else {
>    ...
>    bv->getLyXText()->replaceSelectionWithString(...);
>    ...
> }

I admit I don't understand this. Is this division in seperate branches a
consequence of the "main lyx text" not being a "real inset"?

> And you have to implement all the needed functions. IMO this should not be
> done right now.

Decent S&R is a long standing item not only in my wish list... But it might
stay there a bit longer as well...

Andre'

-- 
Andr� P�nitz .............................................. [EMAIL PROTECTED]

Reply via email to