OR-280 Nudge new installs to run the demo experiment - #338
Merged
Merged
Conversation
Analytics for Sep 9-12 installs showed the largest funnel leak between "onboarding complete" and "first message sent" (43.8%). The demo welcome steered users to read the recorded sessions rather than send anything. - Rename the welcome's primary button to "Run the demo experiment" and hand focus to the composer, which already holds the run-experiment prompt. - Keep that prefill on offer until the user has sent anything in the demo, detected by a recorded session's active leaf moving off its seeded message or a new session appearing, instead of only until the welcome closes. - Show a dismissible hint above the composer explaining the 200-step probe, with a ring on the send button while the prefilled prompt is untouched. - Open only the experiments tab on the first demo visit. - Default the right pane to a quarter of the window (max 480px) on installs with no saved layout, and share that default with the projects home so it is not overwritten before the demo opens. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
- Relabel the welcome button "Try the demo experiment": the click only focuses the prefilled prompt, so "Run" over-promised. - Style the composer hint as a neutral surface bubble instead of a tint. - When a demo run starts, show a second dismissible hint above the composer with a Logs button that opens that run's logs; it clears when the run leaves the running state. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…riment-nudge # Conflicts: # ui/dist/assets/index-BJLMzvdf.css # ui/dist/assets/index-BJUUwp49.js # ui/dist/assets/index-CQbtnwiZ.js # ui/dist/assets/index-mNCWAddR.js # ui/dist/index.html
- Seed the running-run hint from the runs baseline so it survives a reload or a return to the demo project mid-run. - Announce the hint to assistive tech via a polite live region and return focus to the composer on dismiss, matching the prompt hint. - Share the hint bubble classes, drop the redundant Logs tooltip key, and use the idiomatic Arabic phrasing for the tap instruction. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Sep 9–12 install analytics showed the largest funnel leak between "onboarding complete" and "first message sent" (43.8%). The demo welcome pushed users to read the recorded sessions rather than send anything. This PR makes the first message the obvious next step.
explore_demo.activeLeafIdagainst its seeded leaf (DEMO_SEEDED_LEAF_IDS) instead of only looking at session ids.aria-describedby; dismiss returns focus to the composer.ui/src/panelLayout.tsand both callers use it. Existing installs keep their saved width.tour_explore_demorenamed totour_run_demo_experimenteverywhere.ui/distrebuilt on the merged tree.Not changed: the
demoOverviewLeadinglogic inApp.tsxthat pins the overview artifact first when present. Nothing opens that tab automatically anymore, so it only applies to layouts saved by older builds or a manually opened overview. Left for a separate cleanup.Test plan
node ui/scripts/check-i18n.mjs,node ui/scripts/check-styles.mjs, UI typecheck,ui/tests(162 pass)cargo fmt --check,cargo clippy --all-targets -D warnings,cargo build --locked,cargo test --locked(830 pass) on the merged tree🤖 Generated with Claude Code