Jürgen Spitzmüller <juer...@spitzmueller.org> writes:
> Yes, I thought about this as well, but I'd rather not do this in branch now
> (even though this is definitely the way to go in the long term). There are
> some cases where the context menu might act surprisingly (e.g. if a
> collapsable nests a collapsable at pos 0), but IMO this is bearable.
OK.
> So the updated method would look like this:
> if (inset) {
> if (inset->isActive()) {
Probably
if (inset && inset->isActive()) {
instead.
The patch looks fine to me. Nevertheless, I think there is a need to
somehow re-think this contextual menu stuff (but I do not know how).
JMarc
