Skip to content

Sphurthy fix: scrolling in all events page#5030

Open
sphurthy wants to merge 3 commits into
developmentfrom
sphurthy-fix-scrolling-in-all-events-page
Open

Sphurthy fix: scrolling in all events page#5030
sphurthy wants to merge 3 commits into
developmentfrom
sphurthy-fix-scrolling-in-all-events-page

Conversation

@sphurthy

Copy link
Copy Markdown
Contributor

Description

image

Related PRS (if any):

None.

Main changes explained:

  • Updated src/components/CommunityPortal/CPDashboard.jsx to replace the Branches / Themes / Categories filter dropdowns from native select to react-select.
  • Configured each react-select dropdown with menuShouldBlockScroll={false} so opening the dropdown does not lock page scrolling. This directly addresses the “can’t scroll while dropdown is active” UX issue on the “All Events” page.
  • Left the dropdowns as current placeholders (no real option data yet), so behavior change is focused purely on scroll locking.

How to test:

  1. Go to the “All Events” page: https://dev.highestgood.com/communityportal
  2. In the “Search Filters” sidebar, open any dropdown:
  3. Branches (or Themes, Categories)
  4. While the dropdown menu is open:
  5. Try scrolling the page (mouse wheel / trackpad).
  6. Confirm the page continues to scroll normally.
  7. Close the dropdown and repeat with another filter dropdown.
  8. Basic regression check:
  9. Ensure the dropdown still opens/closes as expected and the rest of the page remains interactive.

Screenshots or videos of changes:

image

Note:

None.

@netlify

netlify Bot commented Mar 20, 2026

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 823cb8a
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a3aff33ff0db0000838b873
😎 Deploy Preview https://deploy-preview-5030--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@sonarqubecloud

Copy link
Copy Markdown

@sphurthy sphurthy changed the title fix: scrolling in all events page Sphurthy fix: scrolling in all events page Apr 23, 2026
@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Jun 18, 2026
DeepighaJ
DeepighaJ previously approved these changes Jun 18, 2026

@DeepighaJ DeepighaJ left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested the Branches, Themes, and Categories filter dropdowns. Verified that the page remains scrollable while dropdown menus are open and after they are closed. Functionality is working as expected. Approving this PR.

Image Image Image

kzou55
kzou55 previously approved these changes Jun 18, 2026

@kzou55 kzou55 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Sphurthy,

I ran and tested the current branch locally.

I can verify that scrolling can still be done when the 3 filter's(branches, themes, categories) drop down menu is open.

Recording.2026-06-18.174254.mp4

@HemanthNidamanuru HemanthNidamanuru left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Sphurthy,

Tested this PR. Everything is working fine, the page scrolls normally while filter dropdowns are open, and dropdown open/close behavior works as expected.

Image Image

@sphurthy sphurthy dismissed stale reviews from HemanthNidamanuru, kzou55, and DeepighaJ via 7268657 June 23, 2026 21:26
@sonarqubecloud

Copy link
Copy Markdown

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

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants