Skip to content

[Catalog] Fix Performers button alignment with toolbar - #1859

Merged
leecalcote merged 1 commit into
layer5io:masterfrom
Maanvi212006:fix/performers-button-alignment
Oct 2, 2026
Merged

leecalcote merged 1 commit into
layer5io:masterfrom
Maanvi212006:fix/performers-button-alignment

Conversation

@Maanvi212006

@Maanvi212006 Maanvi212006 commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Notes for Reviewers

  • Fixes the "Hide/Show Performers" button sitting higher than the search bar and sort dropdown in the Catalog toolbar.
  • The wrapping span is now a flex container with height: 100%, and the button fills it, so it matches the height of adjacent elements.

This PR fixes #1858

Signed commits

  • Yes, I signed my commits.

Summary by CodeRabbit

  • Style
    • Improved the performer section button’s height alignment.

Signed-off-by: Maanvi Chetwani <maanvichetwani21@gmail.com>
@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: b4198ba1-e28a-40d3-823f-135ee8130d47

📥 Commits

Reviewing files that changed from the base of the PR and between 5f6fde0 and d17d184.

📒 Files selected for processing (1)
  • src/custom/PerformersSection/PerformersToogleButton.tsx

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.


📝 Walkthrough

Walkthrough

The Performers section button now fills the height of a full-height flex wrapper. Its label, click handler, styling, and open-state behavior remain unchanged.

Changes

Performers button layout

Layer / File(s) Summary
Set button height
src/custom/PerformersSection/PerformersToogleButton.tsx
The wrapper uses a full-height flex layout. The button height is set to 100%.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix · Severity of issue fixed: Low

Merge Risk: ⚪ Minimal · up to d17d1

No repository render path establishes a user-visible alignment regression from this change, so no merge blocker is identified.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: fixing the Performers button alignment with the toolbar.
Linked Issues check ✅ Passed The change addresses issue #1858. PerformersSectionButton now uses a full-height flex wrapper, and the button uses height: 100%. This makes the button fill the toolbar height and supports vertical…
Out of Scope Changes check ✅ Passed The reported change is limited to src/custom/PerformersSection/PerformersToogleButton.tsx. The wrapper and button height changes directly support the Catalog toolbar alignment objective in issue #18…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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.

@leecalcote
leecalcote merged commit 672bc14 into layer5io:master Oct 2, 2026
6 checks passed
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.

[Catalog] "Hide Performers" button misaligned with search bar and sort dropdown

2 participants