Skip to content

Conversation

@rebelchris
Copy link
Contributor

@rebelchris rebelchris commented Jan 12, 2026

Make the PostNavigation component in BasePostModal sticky to ensure the back button remains visible when scrolling through long posts on mobile devices.

This fixes the issue where users had to scroll all the way back to the top to access the back button. The navigation bar now stays at the top of the viewport while scrolling.

Fixes #5277

Generated with Claude Code

Preview domain

https://claude-issue-5277-20260112-0806.preview.app.daily.dev

Make the PostNavigation component in BasePostModal sticky to ensure the back button
remains visible when scrolling through long posts on mobile devices.

This fixes the issue where users had to scroll all the way back to the top to access
the back button. The navigation bar now stays at the top of the viewport while scrolling.

Fixes #5277

Co-authored-by: Chris Bongers <rebelchris@users.noreply.github.com>
@rebelchris rebelchris requested a review from a team as a code owner January 12, 2026 08:10
@vercel
Copy link

vercel bot commented Jan 12, 2026

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

Project Deployment Review Updated (UTC)
daily-webapp Ready Ready Preview Jan 12, 2026 11:36am
1 Skipped Deployment
Project Deployment Review Updated (UTC)
storybook Ignored Ignored Jan 12, 2026 11:36am

@rebelchris
Copy link
Contributor Author

Screenshot 2026-01-12 at 13 34 22

@claude I think that fix doesn't solve the post page though which would show on the described mobile device.

@claude
Copy link
Contributor

claude bot commented Jan 12, 2026

Claude finished @rebelchris's task in 1m 28s —— View job


Analysis Complete

Thanks for raising this concern! You're absolutely right - let me explain what I found.

Todo List

  • Read the current BasePostModal implementation
  • Check issue Back button issue #5277 for the original problem description
  • Investigate how posts are displayed on mobile (modal vs page)
  • Determine if additional changes are needed
  • Provide recommendations

The Issue

Current PR Fix:

  • Modified BasePostModal.tsx to make PostNavigation sticky (line 105: sticky top-0)
  • This only affects modal views (when posts open in a modal overlay)

What About Mobile Post Pages?

Good news! I found that mobile post pages already have a sticky back button through a different component:

  1. Mobile post pages use GoBackHeaderMobile component (BasePostContent.tsx:45-56)
  2. GoBackHeaderMobile is already sticky! (GoBackHeaderMobile.tsx:78: sticky top-0 z-postNavigation)
  3. It includes a back button that appears on mobile/tablet (hidden on laptop+)

So mobile post pages already have the sticky back button functionality through GoBackHeaderMobile.

Verification Needed

However, if you're seeing the back button scroll away on the mobile post page in your screenshot, there might be:

  1. A CSS specificity issue preventing the sticky positioning
  2. A parent container with overflow: hidden or similar that breaks sticky positioning
  3. The issue might be on a specific viewport size

Could you help me understand:

The code suggests it should work, but if it doesn't in practice, I can investigate further and fix any CSS issues preventing the sticky behavior.


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.

Back button issue

2 participants