Skip to content

Conversation

@tsahimatsliah
Copy link
Member

@tsahimatsliah tsahimatsliah commented Feb 2, 2026

Summary

  • Improve feed layout with full-width cards and reduced gaps
  • Make header fixed instead of sticky for YouTube-like behavior
  • Add hidden scrollbar that shows on hover
  • Improve sidebar collapse animation with synchronized 300ms transitions
  • Adjust sidebar padding and font sizes for consistency
  • Make plus icon always visible in section headers

Test plan

  • Verify feed cards are full-width with reduced gaps
  • Verify header stays fixed on scroll (doesn't move up/down)
  • Verify scrollbar is hidden by default and appears on hover
  • Verify sidebar collapse/expand animation is smooth without glitches
  • Verify sidebar padding and alignment is consistent in expanded and collapsed states

Preview domain

https://feed-layout-improvements.preview.app.daily.dev

tsahimatsliah and others added 2 commits February 2, 2026 08:41
- Remove max-width constraints on feed container for full-width layout
- Reduce default grid gap from gap-8 (32px) to gap-4 (16px)
- Unify page padding to laptop:p-10 on all sides

Co-authored-by: Cursor <[email protected]>
- Make header fixed instead of sticky for YouTube-like behavior
- Add hidden scrollbar that shows on hover
- Improve sidebar collapse animation with synchronized 300ms transitions
- Adjust sidebar padding and font sizes for consistency
- Make plus icon always visible in section headers

Co-authored-by: Cursor <[email protected]>
@vercel
Copy link

vercel bot commented Feb 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Feb 3, 2026 5:42pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Feb 3, 2026 5:42pm

Request Review

@CLAassistant
Copy link

CLAassistant commented Feb 2, 2026

CLA assistant check
All committers have signed the CLA.

- Add aria-label to sidebar toggle button for accessibility
- Fix section title opacity class for collapsed state
- Update test to match renamed "Squads" section

Co-authored-by: Cursor <[email protected]>
- Cast isVisible.current to boolean for aria-expanded
- Remove unsupported variant prop from CreatePostButton

Co-authored-by: Cursor <[email protected]>
grid-template-columns: 100%;
}
.container {
@screen laptopL {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this change intentional? It basically makes all sizes the same then? Before this would take user settings for preference into account.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes I think it's better and makes the product simpler from a settings perspective.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can but then we have to maybe remove the settings wdyt?

Copy link
Contributor

@rebelchris rebelchris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generic:

  • There's a layout jump in sidebar if you collapse/expand it now.

tsahimatsliah and others added 2 commits February 3, 2026 14:53
- Remove density settings from appearance page (hardcode to 'eco')
- Debounce sidebar state change to prevent layout jump during animation
- Use proper link for 'Add to Feeds' button instead of router.push()

Co-authored-by: Cursor <[email protected]>
@tsahimatsliah
Copy link
Member Author

@rebelchris I fixed everything you mentioned.

@idoshamun idoshamun requested a review from rebelchris February 3, 2026 13:42
Reverts the removal of max-width calculation that was causing cards
to stretch too wide on large displays.

Co-authored-by: Cursor <[email protected]>
tsahimatsliah and others added 3 commits February 3, 2026 16:45
Cards now use full width on screens below 2560px, and only get
constrained to 20rem per card on ultra-wide/4K displays.

Co-authored-by: Cursor <[email protected]>
Aligns with design system by using the highest defined breakpoint
(2156px) instead of arbitrary 2560px value.

Co-authored-by: Cursor <[email protected]>
Copy link
Contributor

@rebelchris rebelchris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me now.

@claude do one sanity review to see if anything has breaking changes.

@claude
Copy link
Contributor

claude bot commented Feb 3, 2026

Claude finished @tsahimatsliah's task in 1m 49s —— View job


Sanity Review for Breaking Changes

I've reviewed all 21 changed files. Here's my assessment:


⚠️ Potential Breaking Changes

1. Removed Density Setting from UI (Feed.module.css:5, appearance.tsx)

Severity: Medium - User-Facing Regression

The PR removes the "Density" setting (Eco/Roomy/Cozy) from the appearance settings page:

  • Deleted 44 lines from packages/webapp/pages/settings/appearance.tsx
  • Changed Feed.module.css line 5: now ignores user's spaciness preference
  • Impact: Users who previously customized feed density will lose their preference in the UI
  • Mitigation: The spaciness and setSpaciness are still in SettingsContext (packages/shared/src/contexts/SettingsContext.tsx:49), so no API breakage, but user-facing functionality is removed

Recommendation: Consider if this is intentional product simplification or should be reverted. If intentional, consider migrating existing user preferences.

2. Changed Header Positioning (MainLayoutHeader.tsx:96, MainLayout.tsx)

Severity: Low - Visual Behavior Change

Changed from sticky to fixed positioning:

  • Main layout now has pt-14 laptop:pt-16 (packages/shared/src/components/MainLayout.tsx:199)
  • Banner offset changed from laptop:pt-8 to laptop:pt-24 (MainLayout.tsx:205)
  • Impact: Header stays at top during scroll (YouTube-like) instead of moving with content
  • Risk: Minimal - visual change only, no API breakage

✅ Non-Breaking Changes

3. Sidebar Animation Improvements (FeedContext.tsx, SidebarDesktop.tsx, Section.tsx)

Severity: None - Enhancement

Added 300ms debounced sidebar expansion to prevent layout jumps:

  • New SIDEBAR_TRANSITION_DURATION = 300 constant (FeedContext.tsx:13)
  • Debounced sidebar state with useEffect timeout (FeedContext.tsx:130-136)
  • Synchronized transitions across multiple components
  • Impact: Smoother animations, no breaking changes

4. Scrollbar Styling (base.css)

Severity: None - Visual Enhancement

Added auto-hide scrollbar behavior:

  • Scrollbar hidden by default, shows on hover (packages/shared/src/styles/base.css)
  • Works for both WebKit and Firefox
  • Impact: Pure CSS changes, no functional breakage

5. FeedNav Simplification (FeedNav.tsx)

Severity: None - Code Cleanup

Removed 37 lines of code (packages/shared/src/components/feeds/FeedNav.tsx):

  • Simplified component structure
  • Impact: Internal refactoring, no external API changes

Summary

Breaking Changes: 1 user-facing feature removed (Density setting)

Risk Assessment:

  • Low if density removal is intentional product decision
  • Medium if users expect density settings to persist

Recommendation:
✅ Safe to merge IF the density setting removal is a confirmed product decision. Otherwise, consider:

  1. Restoring the density UI, OR
  2. Announcing the deprecation to users, OR
  3. Providing migration path for users with custom density preferences

All other changes are non-breaking enhancements.


| Branch: feed-layout-improvements

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants