On Thu, Jan 17, 2002 at 11:33:58AM +0100, Juergen Vigna wrote:

> >       Paragraph * par = owner->buffer()->getParFromID(id);
> >       owner->view()->text->setCursor(owner->view(), par, 0);
> 
> Ok the code above is plainly wrong IMO. I'll try to fix it up. We have
> to test first if the found paragraph is inside an inset, if no just proceed
> with the call we have there, if yes we have to unlock an eventually locked
> inset and then call the edit() call of the inset to lock. If we want we
> could put that code into BufferView::lockInset()

I'd rather not spread this single case around the code !
the locking is obscure enough as it is

I don't get what's so wrong about it (remember it would be nice to have this
general cursor feature to allow better workarea scrolling ...)

regards
john

-- 
"Before seeking vengeance, dig two graves."
        - Chinese proverb

Reply via email to