Skip to content

Sphurthy fix vertical spacing labels#4929

Open
sphurthy wants to merge 2 commits intodevelopmentfrom
sphurthy-fix-vertical-spacing-labels
Open

Sphurthy fix vertical spacing labels#4929
sphurthy wants to merge 2 commits intodevelopmentfrom
sphurthy-fix-vertical-spacing-labels

Conversation

@sphurthy
Copy link
Contributor

@sphurthy sphurthy commented Mar 2, 2026

Description

image

Related PRS (if any):

None

Main changes explained:

  • Reduced excess vertical spacing between filter labels and dropdowns on the All Events page for Branches, Themes, and Categories.
  • Added a scoped class in CPDashboard.jsx to target only those three filter rows.
  • Updated CPDashboard.module.css to tighten spacing by lowering label bottom margin and dropdown top margin for the scoped rows only.
  • Kept the update UI-only; no logic, API, or functional behavior was changed.

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
    4.Open Community Portal and go to the All Events page.
  4. Locate the Search Filters section and check Branches, Themes, and Categories.
  5. Confirm the label-to-dropdown gap is visibly tighter and consistent with the intended compact layout.
  6. Verify other filters (for example Dates/Online) still look and behave as before.
  7. Test in both light and dark mode to ensure spacing remains consistent.

Screenshots or videos of changes:

image

@netlify
Copy link

netlify bot commented Mar 2, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 3b7c9e4
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69a5c0efba020f0008dae629
😎 Deploy Preview https://deploy-preview-4929--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 2, 2026

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.

1 participant