feat(layout-v2): sidebar rail rework + Slack-style add rows - #6240
Merged
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.
|
…ll-291988 # Conflicts: # packages/webapp/pages/settings/customization/gamification.tsx # packages/webapp/pages/settings/customization/streaks.tsx
Neither field is read by any consumer of useSidebarShortcutItems; they remain as internal memoized values used by addCatalog/pinPage/isPinned. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
capJavert
reviewed
Jul 28, 2026
capJavert
left a comment
capJavert
left a comment
Contributor
There was a problem hiding this comment.
This branch currently has merge conflicts with the base branch, so it is not eligible for final approval and a code review against the current app would be misleading. Please rebase/resolve the conflicts and rerun CI, then request review again with the resulting diff.
…ll-291988 # Conflicts: # packages/shared/src/components/streak/popup/ReadingStreakPopup.tsx # packages/webapp/pages/settings/customization/streaks.tsx
…xture Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
capJavert
approved these changes
Jul 29, 2026
capJavert
left a comment
capJavert
left a comment
Contributor
There was a problem hiding this comment.
Re-reviewed the current conflict-free diff after the main merges. CI is green and I found no remaining code blocker. Approved; GitHub still marks the branch behind main, so update it before merge if required by branch protection.
tsahimatsliah
added a commit
that referenced
this pull request
Jul 30, 2026
…ates, dnd fixes Iteration pass on the production layout v2 sidebar, on top of #6240. Rail - The daily.dev logo doubles as Home: logo at rest, crossfading to the home glyph on hover/focus; click goes to the For You feed and keeps the extension's feed-reset contract. - New post moved below the rail tabs and became a reorderable rail item. - Rail tabs drag via a DragOverlay ghost with the real element parked in-list as a slot skeleton, so the notifications tab's live anchor is never under the pointer at release and can't natively navigate on drop. - Restored the design-system Bubble on the rail notifications bell. Streak - StreakBadge is a 24px circle matching the other rail glyphs: the ring takes the flame's colour, and read-today fills pink with a white flame. - Restored the weekend auto-freeze / used-freeze tooltips on the 30-day calendar. - Freeze row follows sidebar guidelines: title + subtitle with a neutral Subtle button instead of link-blue text. Panels - Dropped the duplicate Get API Access CTA, plus the Jobs and DevCard rows, from the You panel; Feed settings now uses the filter icon. - Hot Takes no longer false-highlights on the home feed. - /daily resolves to the same Profile panel as the feed. - Compact quest rows route to each quest's own completion surface via a shared questDestinations module. - Shortcuts dock accepts a drop below the last icon. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
tsahimatsliah
added a commit
that referenced
this pull request
Jul 30, 2026
…ates, dnd fixes Iteration pass on the production layout v2 sidebar, on top of #6240. Rail - The daily.dev logo doubles as Home: logo at rest, crossfading to the home glyph on hover/focus; click goes to the For You feed and keeps the extension's feed-reset contract. - New post moved below the rail tabs and became a reorderable rail item. - Rail tabs drag via a DragOverlay ghost with the real element parked in-list as a slot skeleton, so the notifications tab's live anchor is never under the pointer at release and can't natively navigate on drop. - Restored the design-system Bubble on the rail notifications bell. Streak - StreakBadge is a 24px circle matching the other rail glyphs: the ring takes the flame's colour, and read-today fills pink with a white flame. - Restored the weekend auto-freeze / used-freeze tooltips on the 30-day calendar. - Freeze row follows sidebar guidelines: title + subtitle with a neutral Subtle button instead of link-blue text. Panels - Dropped the duplicate Get API Access CTA, plus the Jobs and DevCard rows, from the You panel; Feed settings now uses the filter icon. - Hot Takes no longer false-highlights on the home feed. - /daily resolves to the same Profile panel as the feed. - Compact quest rows route to each quest's own completion surface via a shared questDestinations module. - Shortcuts dock accepts a drop below the last icon. Co-Authored-By: Claude Fable 5 <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
2 participants
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