Skip to content

fix: same style as #1659 for filter tabs in org#1665

Merged
danielroe merged 2 commits intonpmx-dev:mainfrom
jellydeck:fried-filters
Feb 26, 2026
Merged

fix: same style as #1659 for filter tabs in org#1665
danielroe merged 2 commits intonpmx-dev:mainfrom
jellydeck:fried-filters

Conversation

@jellydeck
Copy link
Copy Markdown
Contributor

apply same styles as #1659 in filter tabs for design consistency

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 26, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 26, 2026 11:08am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 26, 2026 11:08am
npmx-lunaria Ignored Ignored Feb 26, 2026 11:08am

Request Review

@jellydeck jellydeck marked this pull request as ready for review February 26, 2026 11:06
@jellydeck jellydeck changed the title fix: same colors as #1659 fix: same style as #1659 for filter tabs in org Feb 26, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 098c2aa and 3bc7b44.

📒 Files selected for processing (1)
  • app/components/Filter/Panel.vue

📝 Walkthrough

Walkthrough

The pull request modifies the styling of the Filter Panel component in app/components/Filter/Panel.vue. The search scope toggle container background class is updated from bg-bg to bg-bg-muted. Additionally, scope buttons are enhanced with border classes and refined styling for both active and inactive states. The active state applies bg-bg-subtle, text-fg, and border-fg-subtle, whilst the inactive state uses text-fg-muted and border-transparent. These changes are purely presentational with no modifications to the component's logic, data flow, or public interface.

Possibly related PRs

Suggested reviewers

  • danielroe
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly relates to the changeset, explaining that styling changes are being applied to filter tabs for design consistency to match PR #1659.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@danielroe danielroe added this pull request to the merge queue Feb 26, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Feb 26, 2026
@danielroe danielroe added this pull request to the merge queue Feb 26, 2026
Merged via the queue into npmx-dev:main with commit 76909e5 Feb 26, 2026
18 checks passed
@jellydeck jellydeck deleted the fried-filters branch February 26, 2026 11:39
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