Skip to content

Kanishk – Finish community calendar full view#4370

Merged
one-community merged 3 commits intodevelopmentfrom
kanishk_finish_calendar_full_view
Dec 22, 2025
Merged

Kanishk – Finish community calendar full view#4370
one-community merged 3 commits intodevelopmentfrom
kanishk_finish_calendar_full_view

Conversation

@kanishkagarwal6101
Copy link
Copy Markdown
Contributor

Description

  • Modernize the community calendar experience so category, location, and status filters immediately narrow the events rendered in both the calendar grid and supporting views
  • Persist the user’s selected day, highlight it inside the calendar, and expose a contextual summary panel that mirrors what will be shown in the modal
  • Bring the “Latest News” sidebar into the shared CSS module system and extend dark/light styling so the full layout remains cohesive in either theme

Related PRs

Main Changes

  • Refactored CommunityCalendar.jsx to memoize the event collection, cache filtered events by date, track the active day, and synchronize the calendar tiles, modal, and new sidebar panel with those states
  • Added a selected-date panel that summarizes how many events are scheduled, lists each event with quick metadata and an on-demand “View full details” action, and gracefully handles empty days
  • Updated CalendarActivitySection.jsx and CommunityCalendar.module.css to use module-scoped classes, re-themed the activity feed, and introduced reusable styles for the selected-date panel, event cards, and tile highlighting across light and dark modes

Testing Steps

  • Manual Navigate to /communityportal/calendar, change each filter, and confirm the calendar tiles and selected-date panel react immediately
  • Manual Click different calendar days (with and without events) to verify the new summary panel content, badge highlighting, and empty-state messaging
  • Manual Open the event modal from either the calendar tile or the “View full details” button and confirm they show the same data
  • Manual Toggle between dark and light themes to ensure the calendar grid, detail panel, and activity list remain readable

Screenshots

Screenshot 2025-11-11 at 12 33 41 PM Screenshot 2025-11-11 at 12 33 56 PM

Notes

  • Event data is still mocked and hard-coded for January 2025; adjust those seed dates or wire up the live feed before release so the default month is populated.

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 11, 2025

Deploy Preview for highestgoodnetwork-dev ready!

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

@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 Dec 4, 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 Kanishk,

I have reviewed your PR locally and have a few change requests. Other than them the rest of the functionality does work as per requirement.
In light mode:
As you can see when the date is selected below the value 31 is not that visible in calendar due to the text being in white color
Screenshot 2025-12-03 at 9 27 01 PM
Similarly for dates with no events as well when we hover on them
Screenshot 2025-12-03 at 9 27 21 PM
In dark mode:
Similarly for dates with no events as well when we hover on them in dark mode as well as the text "Select a different date or adjust the filters to see scheduled events." is not as visible in dark mode due to grey text color
Screenshot 2025-12-03 at 9 27 32 PM
Note: Do let me know if any of these do not come under the requirements

debadyuti23
debadyuti23 previously approved these changes Dec 13, 2025
Copy link
Copy Markdown

@debadyuti23 debadyuti23 left a comment

Choose a reason for hiding this comment

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

Hi @kanishkagarwal6101 I have reviewed your changes. I have clicked on the calendar tiles with events/normal. They are working as expected both in light and dark mode.

Drive link: https://drive.google.com/file/d/1OOQGJIy_Mpb598tfV-Wanzk86yscM8Mp/view?usp=sharing

@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 Kanishk,

I have re-reviewed your PR and thanks for making the changes. All the requirements work as expected.
Screenshot 2025-12-19 at 3 32 50 PM
Screenshot 2025-12-19 at 3 33 03 PM
Screenshot 2025-12-19 at 3 33 13 PM

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit f197457 into development Dec 22, 2025
10 checks passed
Copy link
Copy Markdown
Contributor

@Shravan-neelamsetty Shravan-neelamsetty left a comment

Choose a reason for hiding this comment

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

Hi Kanishk, I reviewed this PR after it was merged into development by navigating to /communityportal/calendar and testing the functionality. The calendar filters (category, location, status) update the calendar tiles immediately as expected. The selected date panel correctly displays event summaries when clicking on days with events, and shows appropriate empty state messages for days without events. The "View full details" button and clicking calendar tiles both open the correct event modal with matching data. Dark mode implementation works well, text visibility on selected dates and hover states is clear in both light and dark modes. The Latest News sidebar integrates smoothly with the CSS modules and maintains consistent theming. The calendar experience feels modern and responsive.
Screenshot 2026-01-10 at 3 58 40 PM
Screenshot 2026-01-10 at 3 58 50 PM

Copy link
Copy Markdown

@Vikas-8055 Vikas-8055 left a comment

Choose a reason for hiding this comment

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

Hi Kanishk, I reviewed this PR locally at /communityportal/calendar. The memoized event collection and state synchronization work correctly across calendar tiles, modal, and sidebar panel. The selected-date panel displays event count and metadata properly (tested with January 10, 2026 showing Event 2 details) with "View full details" button functional. Empty days handled gracefully. The Latest News sidebar and CSS module styling work well in both light and dark modes with good text visibility and tile highlighting. All requirements work as expected.

Screenshot 2026-01-10 at 6 20 34 PM Screenshot 2026-01-10 at 6 20 46 PM

Copy link
Copy Markdown
Contributor

@roshini1212 roshini1212 left a comment

Choose a reason for hiding this comment

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

All listed calendar features have been verified and are functioning correctly. Filter changes on /communityportal/calendar update the calendar tiles and selected-date panel immediately. Selecting different calendar days dynamically refreshes the summary panel, highlights active days, and shows appropriate empty-state messages when no events are present. The event modal displays consistent information whether opened from a calendar tile or through the “View full details” button. Theme toggling between dark and light modes maintains full readability across the calendar grid, detail panel, and activity list. All tests passed successfully.

Image Image

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.

7 participants