Skip to content

Akshith - Improve Data Clarity & Interpretation in the Event Popularity Page#4692

Open
akshith312 wants to merge 7 commits into
developmentfrom
akshith-event-pop-improve
Open

Akshith - Improve Data Clarity & Interpretation in the Event Popularity Page#4692
akshith312 wants to merge 7 commits into
developmentfrom
akshith-event-pop-improve

Conversation

@akshith312
Copy link
Copy Markdown
Contributor

Description

image

Related PRS (if any):

This frontend PR is related to the #3192

Main changes explained:

  • Replaced Generic labels with more descriptive names
  • Used Consistent terminology across the page to improve data clarity

How to test:

  1. check into current branch
  2. do npm install and npm run start:local to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to http://localhost:5173/EventPopularity
  6. verify the visuals and check if the metrics display more details.
  7. Check if hovering displays the details correctly.

Screenshots or videos of changes:

Screen.Recording.2026-01-09.161854.mp4

Note:

Mock Data is being used for the visuals

@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 9, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit fc15608
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a2264b9dc611f000836cb98
😎 Deploy Preview https://deploy-preview-4692--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

sonarqubecloud Bot commented Jan 9, 2026

@akshith312 akshith312 changed the title feat: Added hover info details and better interpretation of data Akshith - Improve Data Clarity & Interpretation in the Event Popularity Page Jan 9, 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 Mar 28, 2026
Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

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

Hi Akshith,

I tested the PR locally. Everything is working fine. The visuals look good, metrics display the details correctly, and hovering works as expected. The only issue is that dark mode is not working.

Image Image Image Image Image

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

I have reviewed your PR locally and though the interpretation is as per requirement there are few UI issues:

  • The y axis title is not visible fully
  • The tool tip in Event attendance is too dark and the time is not visible much
  • The dark mode is not well adapted
Image Image Image Image Image Image Image Image Image Image

@akshith312
Copy link
Copy Markdown
Contributor Author

Hi @HemanthNidamanuru @Anusha-Gali I have fixed the hover issues, the dark mode currently doesn't work because it is implemented from a different PR.

@akshith312 akshith312 requested a review from Anusha-Gali April 28, 2026 02:31
@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 5, 2026

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.

4 participants