On Mon, Mar 29, 2004 at 03:05:02PM +0000, Angus Leeming wrote:
> I'd like to change the name of this LFUN to LFUN_NEXT_INSET_TOGGLE, so
> that LFUN_INSET_TOGGLE becomes an LFUN handled by the insets and not
> visible to the outside world. Ok?
> 
>         case LFUN_INSET_TOGGLE:
>                 cur.clearSelection();
>                 if (!toggleInset(cur))
>                         cur.undispatched();
>                 else
>                         bv->switchKeyMap();
>                 break;
> 
> Can you explain why we invoke switchKeyMap if the inset is not a
> collapable inset?

Can anybody explain what 'switchKeyMap' is good for at all and why we
can't simply run it in bv::update() and remove all these 'manual'
invocations?

> Further, I think that toggleInset should be moved into
> InsetCollapable::priv_dispatch. Again, Ok?

Fine with me.

Andre'

Reply via email to