Amaresh taking over for Deekshith: Landing page for event popularity - #5029
Amaresh taking over for Deekshith: Landing page for event popularity#5029Deekshith010905 wants to merge 4 commits into
Conversation
Deekshith010905
added
the
High Priority - Please Review First
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Deekshith010905
changed the title
HemanthNidamanuru
left a comment
HemanthNidamanuru
left a comment
There was a problem hiding this comment.
Hi Deekshith,
I reviewed your PR locally and the landing page for Event Popularity Analytics is looking good. However, I noticed a couple of issues:
- The hover tooltip is not visible
- The No-show rate is not visible in the chart
rohanrastogi311
left a comment
rohanrastogi311
left a comment
There was a problem hiding this comment.
HI Deekshith,
Well done with this implementation.
Anusha-Gali
left a comment
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Deekshith,
I have reviewed your PR locally and as you can see in the images below
- the pdf in dark mode doesn't show the text
- the gaps between texts is close to the next heading that it own heading/title in
- the hover text is not visible
- the colors in UI are not visible
- not sure why all the text in detailed analytics is a link
rajanidi1999
left a comment
rajanidi1999
left a comment
There was a problem hiding this comment.
Hi,
I tested your PR locally and everything is working as expected.
|
Hi Deekshit |
mahathiganimi
left a comment
mahathiganimi
left a comment
There was a problem hiding this comment.
Hi Deekshith,
I have tested this branch locally. Here are my observations - The tooltip hover does not work in dark mode. Detailed analytics are all in link format, would be nice to have just the 'click to view' to be redirecting to another page.
BosuBose132
left a comment
BosuBose132
left a comment
There was a problem hiding this comment.
I tested PR #5029 locally.
The Event Participation Analytics page loads, and I was able to see the summary metrics, upcoming events section, view buttons, and the event type popularity chart.
However, I noticed a few issues that need to be fixed before approval:
-
The page is not responsive. On smaller/narrower widths, the top navigation links appear stuck together without spacing, and the Upcoming Events controls/buttons become cramped. The "+ Create New" button also wraps awkwardly.
-
In dark mode, the chart hover tooltip does not show the data properly. When hovering over the pie chart, it displays as a blank white tooltip box instead of showing the event/category details.
Please fix the responsive layout and the dark mode tooltip readability/data display issue.
AnshShah-18
left a comment
AnshShah-18
left a comment
There was a problem hiding this comment.
Tested the Event Participation landing page locally. The main sections work as expected, but the tooltip hover. Also, the entire Detailed Analytics cards currently redirect to another page; it would be clearer if only the “View Details” link triggered navigation. Please fix the tooltip hover behavior and consider limiting navigation to the link.
RuthwikHM
left a comment
RuthwikHM
left a comment
There was a problem hiding this comment.
Thanks for fixing this!! Things work fine in light mode but in dark mode the text is not visible at all. See the pictures below.
https://github.com/user-attachments/assets/570a617a-8fc0-4e52-991c-0fd79570f509
amaresh2001
changed the title
|
Redone with this pair: #5453 and OneCommunityGlobal/HGNRest#2311 |
11 participants
Description
Related PRS (if any):
No related PRs
…
Main changes explained:
…
How to test:
2.do npm install and ... to run this PR locally
3.Clear site data/cache
4.log as any user
5.go to the URL http://localhost:5173/communityportal/reports/participation
Screenshots or videos of changes:
Landing.Page.for.Event.Popularity.mp4
Note:
Include the information the reviewers need to know.