Skip to content

Conversation

@naiduajaykumar
Copy link
Contributor

@naiduajaykumar naiduajaykumar commented Jan 15, 2026

Description

Fixed the dark theme for input fields and all components in Weekly Summaries Report Page.

Related PRS (if any):

Not related to any PR

How to test:

  1. check into current branch
  2. check into current branch
  3. do yarn install and run this PR locally
  4. Clear site data/cache
  5. log as admin user
  6. go to dashboard→ Reports → Weekly Summaries Report

Screenshots or videos of changes:

Recording.2026-01-14.195133.mp4

@netlify
Copy link

netlify bot commented Jan 15, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit c40180a
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/696ebfda2be4c5000899e35b
😎 Deploy Preview https://deploy-preview-4713--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.

@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 Jan 15, 2026
Copy link

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

I have reviewed your PR locally and most of the issues do work as per requirement, there is "Select Filter" which is not adapted to dark mode.
Issue
Screenshot 2026-01-15 at 7 20 45 PM
Works as expected
Screenshot 2026-01-15 at 7 20 00 PM
Screenshot 2026-01-15 at 7 20 09 PM
Screenshot 2026-01-15 at 7 20 15 PM
Screenshot 2026-01-15 at 7 20 26 PM
Screenshot 2026-01-15 at 7 21 09 PM

Copy link

@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 ajay i have reviewed and tested your pr the dark theme works for all input fields and all components

Screenshot 2026-01-17 at 12 20 35 AM Screenshot 2026-01-17 at 12 23 07 AM Screenshot 2026-01-17 at 12 21 22 AM Screenshot 2026-01-17 at 12 21 05 AM

Copy link

@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.

Hey Ajay,

Well done with this implementation. All functional.

PR 4713 Screenshot PR 4713 Screenshot

Copy link

@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 Ajay,
i have tested this pr locally and everything is functional.
Screenshot 2026-01-17 at 2 24 15 PM
Screenshot 2026-01-17 at 2 28 12 PM
Screenshot 2026-01-17 at 2 28 26 PM

@sonarqubecloud
Copy link

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