Skip to content

fix: mobile layout nits on changelog detail pages#90

Merged
sergical merged 9 commits intomainfrom
fix/mobile-changelog-detail-layout
May 6, 2026
Merged

fix: mobile layout nits on changelog detail pages#90
sergical merged 9 commits intomainfrom
fix/mobile-changelog-detail-layout

Conversation

@sergical
Copy link
Copy Markdown
Member

@sergical sergical commented May 6, 2026

Summary

  • Move share links next to the "Changelog" back button on mobile for better distribution
  • Lower "Copy page" button z-index so it doesn't overlap the sticky header when scrolling
  • Remove empty TOC sidebar so article content is full width

Test plan

  • Open a changelog detail page on mobile — share buttons next to back link
  • On desktop — share buttons stay in metadata strip
  • Scroll down — copy page button goes behind the sticky header
  • Article content spans full container width

🤖 Generated with Claude Code

sergical and others added 4 commits May 5, 2026 20:00
Move share links next to the back button for better mobile distribution.
Lower copy page button z-index so it doesn't overlap the sticky header.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…n desktop

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 6, 2026

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

Project Deployment Actions Updated (UTC)
sentry-changelog Ready Ready Preview, Comment May 6, 2026 1:05am

Request Review

- Remove Reset button (users can just clear the input)
- Move search input from hero to sidebar
- Always render CopyChangelogButton so RSS link doesn't jump
- Update skeleton views to match actual rendered layout

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The changelog list doesn't render images or tags, so the skeleton
shouldn't show placeholders for them.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sergical sergical marked this pull request as ready for review May 6, 2026 00:42
Comment thread src/client/components/copyPageButton.tsx
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5ebe2c2. Configure here.

Comment thread src/app/changelog/[slug]/loading.tsx
Comment thread src/app/changelog/loading.tsx Outdated
- Fix inverted responsive visibility on detail page share skeleton
- Replace invalid Tailwind class w-22 with w-24

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sergical sergical merged commit 5e150e5 into main May 6, 2026
12 checks passed
@sergical sergical deleted the fix/mobile-changelog-detail-layout branch May 6, 2026 01:07
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.

2 participants