chore: remove sidebar in onboarding

[joshsny]

Problem

From looking at recordings, users get very distracted during the onboarding flow by the sidebar and all the products there

Changes

• Remove the sidebar on the products and onboarding flow to make things less complicated
• Fix dark mode on plain pages

Does this work well for both Cloud and self-hosted?

Yes

How did you test this code?

Ran it locally

[greptile-apps]

PR Summary

This PR removes the sidebar during onboarding flows to reduce user distraction, while also fixing dark mode behavior on plain pages and improving layout consistency.

• Modified frontend/src/scenes/scenes.ts to use layout: 'plain' for Products and Onboarding scenes instead of previous app-raw layout
• Added height constraint (h-full) in frontend/src/scenes/products/Products.tsx for proper vertical centering
• Simplified dark mode logic in themeLogic.ts by removing sceneConfig?.layout === 'plain' condition
• Replaced BridgePage component with direct flexbox styling in Wizard.tsx for cleaner layout
• Added padding classes in Onboarding.tsx to compensate for sidebar removal while maintaining proper spacing

_{6 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[posthog-bot]

📸 UI snapshots have been updated

20 snapshot changes in total. 0 added, 20 modified, 0 deleted:

• chromium: 0 added, 16 modified, 0 deleted (diff for shard 1, diff for shard 2)
• webkit: 0 added, 4 modified, 0 deleted (diff for shard 2)

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[github-actions]

Size Change: -114 B (0%)

Total Size: 13.6 MB

ℹ️ View Unchanged

Filename	Size	Change
frontend/dist/toolbar.js	13.6 MB	-114 B (0%)

_{compressed-size-action}

[posthog-bot]

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

• chromium: 0 added, 0 modified, 0 deleted
• webkit: 0 added, 4 modified, 0 deleted (diff for shard 2)

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[posthog-bot]

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

• chromium: 0 added, 0 modified, 0 deleted
• webkit: 0 added, 4 modified, 0 deleted (diff for shard 2)

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[posthog-bot]

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

• chromium: 0 added, 0 modified, 0 deleted
• webkit: 0 added, 4 modified, 0 deleted (diff for shard 2)

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[posthog-bot]

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

• chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
• webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[zlwaterfield]

This is a great change but I wanted to note that we did have something like this in the past and users were annoyed they were "blocked". Is there a way to run this as experiment? Or are you just going to watch the metrics and replays closely after merging?

[joshsny]

This is a great change but I wanted to note that we did have something like this in the past and users were annoyed they were "blocked". Is there a way to run this as experiment? Or are you just going to watch the metrics and replays closely after merging?

Yeah going to stare at replays for a long time to make sure users don't hate it and quit, if they do, we can trial a "skip onboarding" button - but I'd rather avoid that for now to see if this encourages users to just stick to the intended onboarding flow.

This is particularly important for additional projects in an org, where it's annoying to go through things again.

[posthog-bot]

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

• chromium: 0 added, 1 modified, 0 deleted (diff for shard 2)
• webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[posthog-bot]

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

• chromium: 0 added, 2 modified, 0 deleted (diff for shard 1, diff for shard 2)
• webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[posthog-bot]

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

• chromium: 0 added, 1 modified, 0 deleted (diff for shard 1)
• webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[posthog-bot]

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week. If you want to permanentely keep it open, use the waiting label.

[posthog-bot]

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week. If you want to permanentely keep it open, use the waiting label.

[posthog-bot]

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

• chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
• webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[posthog-bot]

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week. If you want to permanentely keep it open, use the waiting label.

[posthog-bot]

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

• chromium: 0 added, 4 modified, 0 deleted (diff for shard 4)
• webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.