Skip to content

Namitha: Fixed number alignment in Most Popular Event chart and convert to CSS modules#4588

Merged
one-community merged 4 commits intodevelopmentfrom
fix/event-stats-number-alignment
Feb 27, 2026
Merged

Namitha: Fixed number alignment in Most Popular Event chart and convert to CSS modules#4588
one-community merged 4 commits intodevelopmentfrom
fix/event-stats-number-alignment

Conversation

@Namitha7070
Copy link
Copy Markdown
Contributor

Description

(PRIORITY Low) Keerthana: Phase 3 – Number Alignment Differs in “Most Popular Event” Graph
Fixed number alignment issue in the "Most Popular Event" chart where percentage and count values were not properly aligned in a vertical column on the right side as specified in the Figma design. Additionally, migrated the component to use CSS Modules for better style encapsulation.

Related PRS (if any):

Main changes explained:

  1. Updated EvenStats to implement proper grid layout for the chart, ensuring percentage values (e.g., 80% (20/25)) are aligned in a fixed vertical column on the right side of each bar.
  2. Updated EvenStats to import and use CSS Modules syntax
  3. Created [EventStats.module] with camelCase class names following CSS Modules convention
  4. Updated [EventStats.module] to use CSS Grid layout for proper three-column alignment (label | bar | value)
  5. Deleted old [EventStats.CSS] file as it has been replaced by the CSS Module version
  6. Updated all className references in the component to use the new CSS Modules pattern for both light and dark mode styling

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 dashboard→ /communityportal/reports/event/personalization
  6. verify function “A” (feel free to include screenshot here)
  7. verify this new feature works in [dark mode] and smaller screens

Screenshots or videos of changes:

Week.20.mp4
Screenshot 2025-12-19 145032 Screenshot 2025-12-19 145101

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 19, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 4d82267
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/699f96b636e095000890efb4
😎 Deploy Preview https://deploy-preview-4588--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

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 Namitha,

I have reviewed your PR locally and though the functionality works as requirements in both the modes. The tiles are not visible in responsiveness below 768px.

4588.mp4
Screenshot 2026-01-23 at 12 11 25 AM

Copy link
Copy Markdown

@rohanrastogi311 rohanrastogi311 left a comment

Choose a reason for hiding this comment

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

Hi Namitha,

Wel done. Fully functional on full screen.

PR 4588 Screenshot 1 PR 4588 Screenshot 2

@Namitha7070
Copy link
Copy Markdown
Contributor Author

Screenshot 2026-02-20 163900 > Hi Namitha, > > I have reviewed your PR locally and though the functionality works as requirements in both the modes. The tiles are not visible in responsiveness below 768px. > > 4588.mp4 >

Screenshot 2026-01-23 at 12 11 25 AM

Hi Anusha, I have fixed the page responsiveness below 768px. Can you review it?

Recording.2026-02-20.164238.mp4
Screenshot 2026-02-20 163900

Anusha-Gali
Anusha-Gali previously approved these changes Feb 22, 2026
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 Namitha,

Thanks for fixing the issue. I am able to see the KPI's in smaller screens.

Image Image Image

@sonarqubecloud
Copy link
Copy Markdown

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 9ecb141 into development Feb 27, 2026
10 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.

4 participants