Skip to content

Prem UI overlapping issue#4239

Merged
one-community merged 6 commits intodevelopmentfrom
Prem-UI-overlapping-issue
Nov 27, 2025
Merged

Prem UI overlapping issue#4239
one-community merged 6 commits intodevelopmentfrom
Prem-UI-overlapping-issue

Conversation

@Prem203
Copy link
Copy Markdown
Contributor

@Prem203 Prem203 commented Oct 21, 2025

Description

image

Related PRS (if any):

This frontend PR is not related to any backend PR.

Main changes explained:

  • Updated CPDashboard.jsx to fix UI allignment.
  • Changed css file name to module.css

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to /communityportal
  6. verify the section under search filters

Screenshots or videos of changes:

image

Note:

Dark mode was not specified in the description.

@Prem203 Prem203 added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Oct 21, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 21, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 09ed377
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6927b76158a4100008ad1b84
😎 Deploy Preview https://deploy-preview-4239--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
Copy Markdown

Aswin20010
Aswin20010 previously approved these changes Oct 21, 2025
Copy link
Copy Markdown

@Aswin20010 Aswin20010 left a comment

Choose a reason for hiding this comment

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

Reviewed PR #4239(Community Portal Dashboard UI Alignment Fix). Verified updated alignment adjustments in CPDashboard.jsx. Checked layout consistency under search filters and overall visual stability. No UI or functional regressions found — approved the PR.

Screenshot 2025-10-21 at 12 27 23 PM

@apoorvajainrp21
Copy link
Copy Markdown
Contributor

I tested the above functionality: 1. None of the drop-down buttons present options on selection. 2. No change in events after changing filters. 3. Show past events button does not yield anything. I have attached screenshots of my testing:
Screenshot 2025-10-22 145031
Screenshot 2025-10-22 145134

Aditya-gam
Aditya-gam previously approved these changes Oct 24, 2025
Copy link
Copy Markdown
Contributor

@Aditya-gam Aditya-gam left a comment

Choose a reason for hiding this comment

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

This PR successfully addresses UI alignment issues in the Community Portal dashboard through a well-executed migration to CSS Modules. The conversion from CPDashboard.css to CPDashboard.module.css follows React best practices for style encapsulation, preventing class name conflicts in the larger application.

  • All className references correctly updated to bracket notation (styles['class-name'])
  • Added filter-section-divider wrapper with flexbox layout to resolve the filter section alignment issue
  • Removed fluid prop from Container, improving layout constraints with explicit max-width: 1400px
  • Consistent formatting improvements (multi-line img attributes)
TestVideo.mov

Concerns:

  • The CSS file contains global body styles, which defeats CSS Module scoping benefits
  • Some commented-out CSS properties (/* margin-bottom: 20px; */) should be removed for cleaner code

@hemanthvenkat
Copy link
Copy Markdown

Verified PR as Admin user on /communityportal. The Search Filters section displays correctly — all filter options (Dates, Online, Branches, Themes, Categories) render properly with functional dropdowns and calendar input. Layout, contrast, and alignment look consistent across the interface. Overall, the feature works as expected.

image

@beblicarl
Copy link
Copy Markdown
Contributor

This UI looks as intended

image

@beblicarl beblicarl self-requested a review November 7, 2025 12:01
beblicarl
beblicarl previously approved these changes Nov 7, 2025
Anusha-Gali
Anusha-Gali previously approved these changes Nov 23, 2025
Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Prem,

I have reviewed your PR locally and all filter options (Dates, Online, Branches, Themes, Categories) render properly with functional dropdowns and calendar input. Layout, contrast, and alignment look consistent across the interface. But None of the drop-down buttons present various options on selection, similarly there is no change in events after changing filters nor the past events button does anything. Since this is a UI PR i will be approving but a bit more functionality would have been great.
https://github.com/user-attachments/assets/5ff08604-8d87-4b27-b9a5-cfdf85d75669

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@sonarqubecloud
Copy link
Copy Markdown

@one-community one-community merged commit 9f689fe into development Nov 27, 2025
10 checks passed
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.

8 participants