Skip to content

[Customer Portal][Web] Fix UI bug in Multi-select Filters - #777

Merged
shayanmalinda merged 1 commit into
wso2-open-operations:mainfrom
Rashmika998:main
May 30, 2026
Merged

shayanmalinda merged 1 commit into
wso2-open-operations:mainfrom
Rashmika998:main

Conversation

@Rashmika998

@Rashmika998 Rashmika998 commented May 30, 2026 •

Copy link
Copy Markdown
Contributor

Description

  • Fix UI bug in the multi-select filters component.

Summary by CodeRabbit

  • Style
    • Enhanced spacing on multi-select filter controls for medium and larger screens to improve visual alignment and layout consistency in the list view.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 30, 2026 •

Copy link
Copy Markdown
Contributor

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 01a183dd-fae1-49e3-8f56-d4f91f89c08e

📥 Commits

Reviewing files that changed from the base of the PR and between e47adf6 and ee1be5a.

📒 Files selected for processing (1)
  • apps/customer-portal/webapp/src/components/list-view/ListFilters.tsx

📝 Walkthrough

Walkthrough

This PR adds responsive top margin styling to the multi-select filter FormControl in the ListFilters component. The FormControl now applies a 1-unit top margin on medium breakpoints and larger, improving visual spacing between filter elements.

Changes

Multi-select filter spacing adjustment

Layer / File(s) Summary
FormControl margin adjustment
apps/customer-portal/webapp/src/components/list-view/ListFilters.tsx
Multi-select filter FormControl receives responsive top margin spacing via sx={{ marginTop: { md: 1 } }} on medium+ breakpoints.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

Type/Improvement

Suggested reviewers

  • cloby99

Poem

🐰 A margin here, responsive and neat,
Makes filters align, visually sweet!
On medium screens, the space finds its place,
One tiny line, improves the whole space. ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is minimal and incomplete, missing most required sections from the template including Purpose/Goals, Approach (with no UI screenshot), User stories, Release notes, and other standard sections. Expand the description to include Purpose (with issue links), Goals, detailed Approach with UI screenshot, User stories, Release notes, and other applicable template sections for clarity and documentation standards.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: fixing a UI bug in multi-select filters, which aligns with the code modification adding marginTop styling to the FormControl.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@shayanmalinda
shayanmalinda merged commit 8ecf750 into wso2-open-operations:main May 30, 2026
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants