Skip to content

fix(conversations): Restore side-by-side layout for platform option dropdown#116272

Merged
priscilawebdev merged 1 commit into
masterfrom
priscilawebdev/fix/conversations-conversations-onboarding-with-selector-layout
May 27, 2026
Merged

fix(conversations): Restore side-by-side layout for platform option dropdown#116272
priscilawebdev merged 1 commit into
masterfrom
priscilawebdev/fix/conversations-conversations-onboarding-with-selector-layout

Conversation

@priscilawebdev
Copy link
Copy Markdown
Member

Restore the inline with <SDK> layout on the conversations onboarding panel.

In #116082 the <Flex gap="md" align="center" wrap="wrap" paddingBottom="md"> wrapper around PlatformOptionDropdown was replaced by a <Stack gap="md"> that now wraps the dropdown, the introduction, and the guided steps together. PlatformOptionDropdown returns a Fragment containing the "with" text node and the OptionControl dropdown(s) as siblings — so the Stack hoisted them as separate vertical items and the label ended up above the selector.

Wrap just the dropdown row in a <Flex gap="md" align="center" wrap="wrap"> inside the Stack. The Stack's own gap="md" already handles spacing to the next row, so the previous paddingBottom="md" is no longer needed.

Refs GH-116082

…ropdown

The recent scraps primitives refactor replaced the `<Flex>` wrapper around
`PlatformOptionDropdown` with a `<Stack>` that now wraps the whole onboarding
content. Because `PlatformOptionDropdown` returns a Fragment with the "with"
text and the dropdown as siblings, the Stack hoisted them as separate vertical
items, breaking the inline row.

Wrap just the dropdown in a `<Flex gap="md" align="center" wrap="wrap">` inside
the Stack so the label and selector render on the same line again.

Refs GH-116082
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.58%

@priscilawebdev priscilawebdev marked this pull request as ready for review May 27, 2026 06:59
@priscilawebdev priscilawebdev requested a review from a team as a code owner May 27, 2026 06:59
@priscilawebdev priscilawebdev enabled auto-merge (squash) May 27, 2026 07:01
@priscilawebdev priscilawebdev merged commit fb60bc3 into master May 27, 2026
76 checks passed
@priscilawebdev priscilawebdev deleted the priscilawebdev/fix/conversations-conversations-onboarding-with-selector-layout branch May 27, 2026 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants