Make deliberate trips to Home land on the home surface - #130
Merged
Conversation
The index route redirected into a default-project draft on every visit, which made "Go to Home" and the settings back button teleport the user into a fresh composer they didn't ask for (and quietly mint drafts as a side effect). The redirect is launch behaviour, not a rule about the route: the launch visit still restores the last thread or opens the default draft exactly as before, and every visit after that renders the pick-up-a-thread canvas that until now was only reachable with zero projects.
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
The launch-vs-deliberate flag was only consumed inside the index route's own redirect, but the launch does not necessarily pass through `/` at all - the bootstrap welcome payload and a desktop deep link land directly on a thread route. In those boots a later "Go to Home" was mistaken for the launch visit and either bounced straight back to the thread or stranded a blank canvas. Being on any thread or draft now marks the launch moment consumed (via the route recorder, which already watches exactly that), and the flag lives in its own module so both routes can reach it. Verified end to end in the dev harness: launch lands in the draft, reload restores it, Go to Home shows the pick-up-a-thread canvas and stays, New thread from Home reuses the empty draft, Home is repeatable.
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.
Approved UX change: launch still lands you in your work (restore or default draft, unchanged), but 'Go to Home' and settings-back now show the pick-up-a-thread canvas - recent threads, New thread button, Search - instead of teleporting into a fresh draft. The launch-vs-deliberate distinction reuses the once-per-load restore seam from #128. Full web unit (2101) and browser (390) suites green.