Skip to content

Namitha: Fix calendar visibility, booked date indicators, and dark mode implementation#4544

Open
Namitha7070 wants to merge 1 commit intodevelopmentfrom
namitha/calendar-fix-dark-mode
Open

Namitha: Fix calendar visibility, booked date indicators, and dark mode implementation#4544
Namitha7070 wants to merge 1 commit intodevelopmentfrom
namitha/calendar-fix-dark-mode

Conversation

@Namitha7070
Copy link
Copy Markdown
Contributor

Description

(PRIORITY Medium) Jae/Sudarshan- Fix Calendar Not Showing on Activities Registration Page , Highlight Booked Days and Fix Dark Mode text and colours

  1. Ensure the calendar component renders on the main page.
  2. Fix any missing data or props that prevent the calendar from loading.
  3. Verify calendar functions such as date navigation and event display work properly.
  4. Check styling or layout issues that may be hiding the calendar.
  5. Add a clear visual indicator (e.g., highlight, badge, or icon) on days that are booked so users can easily see which dates are already taken.
  6. Fix the dark mode implementation of text fields

Related PRS (if any):

Main changes explained:

  1. Calendar: Enabled react-calendar on the Register page with selection and navigation.
  2. Booked Days: Highlighted with a red dot and light-red background; added a small legend.
  3. Dark Mode: Used Redux [darkMode]
  4. CSS Modules: Converted page styles
  5. React Calendar Styles: Used :global(...) in CSS modules so calendar classes style correctly.
  6. Responsiveness: Added wrap and media queries to prevent calendar cropping on small screens.
  7. Lint Fixes: Removed duplicate CSS selectors to satisfy pre-commit stylelint.

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/activities/Register/1
  6. verify whether the Calendar renders properly in the page and also test the Dark Mode.

Screenshots or videos of changes:

Week.19.mp4

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 12, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 5be7dc3
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/693ba2044ac4f800099b1303
😎 Deploy Preview https://deploy-preview-4544--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, the calendar does display as per requirement in both modes. But i was unable to verify the point "Verify calendar functions such as date navigation and event display work properly" and also what about if dates are available to book?
Screenshot 2026-01-22 at 11 58 29 PM
Screenshot 2026-01-22 at 11 58 40 PM

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,

Well done with this implementation.

PR 4544 Screenshot PR 4544 Screenshot 2

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

Thanks for working on this. Here's what I found after testing locally:

Working:

  1. The calendar is rendering correctly on the main page.
  2. Date navigation is working as expected.

Not working / needs attention:

  1. I wasn't sure how to verify the event display properly. Could you share some steps or context on how to test this?
  2. Dates where tickets are available need some kind of visual indication so users can easily tell them apart from fully booked or unavailable days.
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.

5 participants