refactor(discover): map-first, drop Table+Graph toggle#67
Open
lantos1618 wants to merge 1 commit into
Open
Conversation
…plore All The /hire page had FOUR control axes stacked on a single screen: header input + JD link, My Network ↔ Explore All, Following/Followers/ Similar filters, AND Table/Graph/Map view. Way too many decisions before the user sees the value. Cuts: - Drop the Table/Graph/Map view toggle entirely. Explore All always renders the SemanticMap (avatars on UMAP). My Network always renders the NetworkGraph. - Delete the now-unused NetworkTable and ExploreAllTable components. - Default landing to Explore All for every visitor (signed in or not). The map IS the value prop; show it before any input is required. ?seed=<u> still forces network mode for the OAuth round-trip from /hire/<u>'s "Find similar" CTA. - Pre-fill the seed input with the signed-in user's handle so flipping to My Network is still one click — without hijacking the landing. Why: value-first funnel. The user lands, sees a map of every analyzed dev with avatars, and clicks into the one they want. No mode-picking, no view-picking — just the map. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
1 participant
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
/hirehad four control axes on one screen: header input, My Network ↔ Explore All, Following/Followers/Similar pills, AND Table/Graph/Map toggle. Cuts the view toggle entirely.?seed=<u>still forces network mode for the OAuth round-trip from/hire/<u>'s "Find similar" CTA.Net: −218 / +48 lines. NetworkTable and ExploreAllTable deleted.
Test plan
/hire→ sees Explore All map immediately, no mode picker required./hire→ also sees Explore All by default; their handle pre-fills the seed input./hire?seed=torvalds→ forces network mode as before.🤖 Generated with Claude Code