Skip to content

Radia : Fix - Loss Tracking calendar picker feature#5371

Open
Radia731 wants to merge 1 commit into
developmentfrom
radia-loss-tracking-calendar-fix
Open

Radia : Fix - Loss Tracking calendar picker feature#5371
Radia731 wants to merge 1 commit into
developmentfrom
radia-loss-tracking-calendar-fix

Conversation

@Radia731

@Radia731 Radia731 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor
Screenshot 2026-07-05 at 1 54 58 AM

Description

This PR resolves the high-priority bug regarding the broken calendar feature on the Loss Tracking line graph. The previous implementation used a native <input type="month"> which prevented users from entering custom specific dates and caused UI inconsistencies. This PR implements a standard date picker, safely reformats the date strings for the chart's data filtering logic, and resolves lingering CSS linter errors.

Fixes # (Priority High - Loss Tracking Calendar Fix)

Related PRS (if any):

This frontend PR fixes the broken calendar implementation originally introduced in PR #5016.

Main changes explained:

  • Update LossTrackingLineChart.jsx to replace <input type="month"> with a standard <input type="date"> picker. Restored the custom SVG calendar icon JSX. Updated the chartData filtering logic to safely slice standard YYYY-MM-DD string outputs down to YYYY-MM so they correctly filter the chart's raw data.
  • Update LossTrackingLineChart.module.css to update all styling selectors from [type="month"] to [type="date"]. Reactivated the custom SVG icon styling and expanded the invisible native calendar picker indicator to 100% width/height so clicking anywhere in the input box opens the calendar.

How to test:

  1. check into current branch (radia-loss-tracking-calendar-fix)
  2. do npm install and npm start to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to BM Dashboard → Total Construction Summary
  6. Scroll down and click the "Loss Tracking ∨" accordion to expand the section.
  7. Verify that clicking the "Start Date" or "End Date" boxes now opens a standard calendar that allows you to pick specific custom days
  8. Verify that selecting a specific date range successfully filters the line graph data.
  9. Set a Start Date that is after the End Date and verify the red validation error message still appears.
  10. verify this new feature works and the custom calendar icon is visible in dark mode.

Screenshots or videos of changes:

Screen.Recording.2026-07-05.at.2.06.59.AM.mov
Screenshot 2026-07-05 at 2 04 19 AM Screenshot 2026-07-05 at 2 04 24 AM Screenshot 2026-07-05 at 2 04 32 AM Screenshot 2026-07-05 at 2 04 43 AM Screenshot 2026-07-05 at 2 04 55 AM Screenshot 2026-07-05 at 2 04 59 AM

@netlify

netlify Bot commented Jul 5, 2026

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit fa17d65
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a49f518edbac80007ec56d5
😎 Deploy Preview https://deploy-preview-5371--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud

sonarqubecloud Bot commented Jul 5, 2026

Copy link
Copy Markdown

@Radia731 Radia731 changed the title fix: replaced native month input with standard date picker for loss t… Radia : Fix - Loss Tracking calendar picker feature Jul 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant