From d70b754652521fc1bc15d8463ba88aa58d73d6fd Mon Sep 17 00:00:00 2001 From: Eva Marco Date: Tue, 6 Oct 2026 13:03:38 +0200 Subject: [PATCH] :bug: Lay out token pills in nested groups side by side again (#12120) The nested group wrapper only turned into a wrapping flex row when it had a direct button child. Since the pills are wrapped in the DS tooltip (#11900), the button is no longer a direct child, so pills in nested groups were stacked one per row. Match pills as "any child that is not a group" instead, so the rule no longer depends on the pill's own markup. Also drop the .token-pill rules of this module: CSS modules prefix them with the token_tree scope, so they never matched the pill button. Closes #12099 AI-assisted-by: claude-opus-5-5 --- .../ui/workspace/tokens/management/token_tree.scss | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/frontend/src/app/main/ui/workspace/tokens/management/token_tree.scss b/frontend/src/app/main/ui/workspace/tokens/management/token_tree.scss index 87ef7f2e4d..ee686c5f8e 100644 --- a/frontend/src/app/main/ui/workspace/tokens/management/token_tree.scss +++ b/frontend/src/app/main/ui/workspace/tokens/management/token_tree.scss @@ -22,10 +22,6 @@ margin-block-end: var(--sp-s); } } - - & .token-pill { - flex: 0 0 auto; - } } .node-parent { @@ -37,7 +33,9 @@ padding-inline-start: var(--sp-s); border-inline-start: $b-2 solid var(--color-background-quaternary); - &:has(> button) { + // Match pills as "not a group" so the rule does not depend on the + // pill's own markup, which lives in another module. + &:has(> :not(.node-parent)) { display: flex; flex-wrap: wrap; gap: var(--sp-xs); @@ -50,8 +48,4 @@ margin-block-end: var(--sp-s); } } - - & .token-pill { - flex: 0 0 auto; - } }