feat(layout-v2): sidebar rail rework + Slack-style add rows#6240
Open
tsahimatsliah wants to merge 2 commits into
Open
feat(layout-v2): sidebar rail rework + Slack-style add rows#6240tsahimatsliah wants to merge 2 commits into
tsahimatsliah wants to merge 2 commits into
Conversation
Port of mock-up PR #6218 onto current main. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…ory query - Remove unused useSidebarDragStateValue hook - Remove write-only outsideRailRef in shortcuts dock - Replace inline 30-day streak history query in ReadingStreakPopup with the shared hook; rename it to useReadingStreak30Days to avoid colliding with the existing useReadingHistory hook - Trim unreachable entries from ExploreHubHeader.hubTitles Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
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.
Reworks the v2 dual-sidebar based on the latest design review, plus consistent add-action affordances for the squad/folder/feed panels. All changes are v2-only — the v1 sidebar and the shared
MainSectionare untouched.This is the production-ready port of the design mock-up in #6218, rebased on current
main(keeps theUpvote / Downvotetooltip copy from #6238) with a cleanup pass over dead code.Rail layout (top → bottom)
Panels
/postssub-tabs (Popular, By upvotes, By comments, By date, Best of) + Recent pages, reusingurlToTabfromFeedExploreHeaderso it can't drift.Add affordances (squads / folders / feeds)
SIDEBAR_ADD_TOP_THRESHOLD), so short lists stay clean.Cleanup vs. the mock-up
useSidebarDragStateValuehook and the write-onlyoutsideRailRefin the shortcuts dock.ReadingStreakPopupnow uses the shared 30-day history hook instead of an inline duplicate query; renamed it touseReadingStreak30Daysto avoid colliding with the existinguseReadingHistoryhook.ExploreHubHeader.hubTitles.Verification
typecheck-strict-changedclean;eslint --max-warnings 0clean for changed files.sidebarCategory.spec.ts).🤖 Generated with Claude Code
Preview domain
https://claude-reverent-maxwell-291988.preview.app.daily.dev