Jean-Marc Lasgouttes wrote:
> Angus> Can you explain why we invoke switchKeyMap if the inset is
>>> not Angus> a collapable inset?
>>>
>>> It is some obscure RtL support thing. I do not know more about it.
>
> Angus> It should have a separate LFUN_INSET_RTL_TOGGLE.
>
> No, it rather seems to be a cleanup function that is added in a lot
> of places.
Huh? toggleInset affects only collapsable insets:
case LFUN_INSET_TOGGLE:
cur.clearSelection();
if (!toggleInset(cur))
cur.undispatched();
else
bv->switchKeyMap();
break;
So the RTL stuff is actually applied only when nothing else is
changed. It *can't* be a clean-up; there's nothing to clean ;-)
--
Angus
