Skip to content

Aayush event details popup frontend#4955

Merged
one-community merged 12 commits intodevelopmentfrom
aayush_event_details_popup_frontend
Apr 18, 2026
Merged

Aayush event details popup frontend#4955
one-community merged 12 commits intodevelopmentfrom
aayush_event_details_popup_frontend

Conversation

@aayushshetty12
Copy link
Copy Markdown
Contributor

Description

Screenshot 2026-03-06 201237

Related PRS (if any):

Main changes explained:

  • Updated ActivityList.jsx to add a modal

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/activities
  6. check if the modal is displayed correctly with all details and closes correctly
  7. verify this new feature works in dark mode

Screenshots or videos of changes:

Recording.2026-03-06.200858.mp4

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 7, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit ecd7bc7
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69e33581047ce400082fcdb6
😎 Deploy Preview https://deploy-preview-4955--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

sonarqubecloud Bot commented Mar 7, 2026

Copy link
Copy Markdown

@Vishnu10-hub Vishnu10-hub left a comment

Choose a reason for hiding this comment

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

Checked out the branch and ran the application locally.

Navigated to /communityportal/activities and tested the event details popup.

The modal opens and closes correctly and works in dark mode.

However, the Types dropdown menu box is not displaying in the modal.

Please verify this issue.

Image

@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 Apr 3, 2026
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 Aayush,

Dark mode is not functional.

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

I have reviewed your PR locally and though the modal popup works as per requirement, the type is no more a dropdown and even if a text is typed - the search does not work as per requirement. The page's dark mode appears different then your video reference.

Image Image Image Image Image

@Nirali164
Copy link
Copy Markdown

Hello Ayush,

I have used your branch to check this feature.
However, it has so many issues :

  1. The Enter type is a textbox and not a dropdown, as shown in the video
  2. The clear all button is grey in color, so the text should be bold white or something more visible.
  3. In dark mode, the clear all button is less visible, or I would say it's not appropriate.
  4. In Dark mode, the big issue is that the sub information (Date, Location) of each event is not present.
Screenshot 2026-04-08 at 11 47 04 AM Screenshot 2026-04-08 at 11 48 01 AM Screenshot 2026-04-08 at 11 48 48 AM Screenshot 2026-04-08 at 11 49 24 AM Screenshot 2026-04-08 at 11 49 50 AM

Anusha-Gali
Anusha-Gali previously approved these changes Apr 16, 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 Aayush,

Thanks for fixing the issue. The modal and other features work as per requirement.

Image Image

@sonarqubecloud
Copy link
Copy Markdown

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit e99e570 into development Apr 18, 2026
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.

6 participants