Skip to content

Commit d87937b

Browse files
fix: keep onboarding provider setup top-aligned (Zoo-Code-Org#113)
Co-authored-by: Roomote <roomote@roocode.com>
1 parent 39342a7 commit d87937b

2 files changed

Lines changed: 19 additions & 2 deletions

File tree

webview-ui/src/components/welcome/WelcomeViewProvider.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ const WelcomeViewProvider = () => {
8585

8686
return (
8787
<Tab>
88-
<TabContent className="flex flex-col gap-4 p-6 justify-center">
88+
<TabContent className="flex flex-col gap-4 p-6">
8989
<Brain className="size-8" strokeWidth={1.5} />
9090
<h2 className="mt-0 mb-0 text-xl">{t("welcome:providerSignup.heading")}</h2>
9191

webview-ui/src/components/welcome/__tests__/WelcomeViewProvider.spec.tsx

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,11 @@ vi.mock("../../settings/ApiOptions", () => ({
3434
// Mock Tab components
3535
vi.mock("../../common/Tab", () => ({
3636
Tab: ({ children }: any) => <div data-testid="tab">{children}</div>,
37-
TabContent: ({ children }: any) => <div data-testid="tab-content">{children}</div>,
37+
TabContent: ({ children, className }: any) => (
38+
<div data-testid="tab-content" className={className}>
39+
{children}
40+
</div>
41+
),
3842
}))
3943

4044
// Mock RooHero
@@ -133,6 +137,12 @@ describe("WelcomeViewProvider", () => {
133137
expect(screen.getByTestId("trans-welcome:providerSignup.chooseProvider")).toBeInTheDocument()
134138
})
135139

140+
it("keeps the landing screen centered", () => {
141+
renderWelcomeViewProvider()
142+
143+
expect(screen.getByTestId("tab-content")).toHaveClass("justify-center")
144+
})
145+
136146
it("does not enter auth-in-progress state after clicking 'Get Started' on landing", () => {
137147
renderWelcomeViewProvider()
138148

@@ -172,6 +182,13 @@ describe("WelcomeViewProvider", () => {
172182
expect(screen.getByTestId("trans-welcome:providerSignup.chooseProvider")).toBeInTheDocument()
173183
})
174184

185+
it("top-aligns provider setup content so tall forms remain reachable", () => {
186+
renderWelcomeViewProvider()
187+
navigateToProviderSelection()
188+
189+
expect(screen.getByTestId("tab-content")).not.toHaveClass("justify-center")
190+
})
191+
175192
it("shows API options immediately", () => {
176193
renderWelcomeViewProvider()
177194
navigateToProviderSelection()

0 commit comments

Comments
 (0)