tree: don't nest redundant containers on repeated split by michaeladler · Pull Request #1138 · mahkoh/jay

michaeladler · 2026-07-29T12:00:35Z

Splitting a top-level container that is the sole child of its parent previously wrapped it in yet another single-child container. Repeating this action created a chain of redundant nesting levels, which is especially noticeable (and somewhat irritating) when borders are enabled.

Instead of creating redundant wrapper containers, this change updates the split direction of the existing parent container.

mahkoh · 2026-07-29T12:33:19Z

I have not looked at the code, but this needs to be a configuration so as to not break existing workflows.

michaeladler · 2026-07-30T07:37:36Z

I considered it a bug so I didn't make it configurable initially :) I've made it configurable now.

mahkoh · 2026-07-30T17:29:22Z

LGTM otherwise.

Splitting a toplevel that is the only child of its parent container wrapped it in yet another single-child container. Repeating the action produced a chain of redundant nesting levels. Change the split direction of the existing container instead.

michaeladler force-pushed the fix/nested-containers branch from 64862f5 to 229ec70 Compare July 30, 2026 07:36