Skip to content

fix(feed): add bottom padding to chronological feed filter pills#14310

Merged
dylanjeffers merged 1 commit into
mainfrom
fix/chronological-feed-padding
May 14, 2026
Merged

fix(feed): add bottom padding to chronological feed filter pills#14310
dylanjeffers merged 1 commit into
mainfrom
fix/chronological-feed-padding

Conversation

@dylanjeffers
Copy link
Copy Markdown
Contributor

Summary

  • The web FeedFilters component renders into the Header's bottomBar slot with no padding-bottom, leaving the All Posts / Original Posts / Reposts pills flush against the header's bottom border.
  • Add pb='l' to give the pills breathing room before the page content begins, matching the pattern already used by the trending page's bottomBar and mirroring the existing paddingBottom: spacing.s on the mobile FeedFilters component.

Test plan

  • Open the web app, sign in, navigate to Your Feed.
  • With the Chronological tab selected, verify there is a clear gap between the filter pills (All Posts / Original Posts / Reposts) and the first track tile below — pills no longer sit flush against the header border.
  • Switch to the For You tab and confirm the layout is unchanged (no bottomBar there).
  • Toggle between filter pills and confirm the spacing remains consistent.

🤖 Generated with Claude Code

The web FeedFilters component renders into the Header's bottomBar slot
with no padding-bottom, leaving the All Posts / Original Posts / Reposts
pills flush against the header's bottom border. Add pb='l' to give the
pills breathing room before the page content begins, matching the
pattern used by the trending page's bottomBar and mirroring the existing
paddingBottom on the mobile FeedFilters component.

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

changeset-bot Bot commented May 14, 2026

⚠️ No Changeset found

Latest commit: aa8fc5c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dylanjeffers dylanjeffers merged commit efa8352 into main May 14, 2026
8 checks passed
@dylanjeffers dylanjeffers deleted the fix/chronological-feed-padding branch May 14, 2026 00:18
@github-actions
Copy link
Copy Markdown
Contributor

🌐 Web preview ready

Preview URL: https://audius-web-preview-pr-14310.audius.workers.dev

Unique preview for this PR (deployed from this branch).
Workflow run

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant