Sriamsh - Improve Daily Equipment Log User Experience#4701
Sriamsh - Improve Daily Equipment Log User Experience#4701Sriamshreddy000 wants to merge 2 commits intodevelopmentfrom
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
sayali-2308
left a comment
There was a problem hiding this comment.
The core functionality works perfectly, but there's a CSS issue in dark mode that needs to be addressed before approval.
What Works Well:
- Confirmation Prompt: Yellow banner appears correctly with "Are you sure? This will update equipment availability." message and Cancel/Confirm buttons
- Success Notification: Green "Equipment logs updated successfully!" message displays after confirmation
- Cancel Functionality: Cancel button properly prevents updates
- Data Persistence: Data survives page refresh correctly
- UX Flow: Clear separation between user intent (submit) and execution (confirm)
- No Console Errors: Clean console in both light and dark modes
Issues Found - Dark Mode:
Project Dropdown and Confirmation Prompt text CSS Issue:
- The Project dropdown field appears with white/light background in dark mode
- Makes the dropdown difficult to read and doesn't integrate cleanly with dark mode theme
- This conflicts with the PR requirement: "Made minor UI adjustments to ensure the confirmation banner integrates cleanly with the existing layout"
Fixed the dark mode styling issue for the confirmation banner. It should now display correctly with proper contrast and visibility in dark mode. |
063098b to
2e0cb8e
Compare
|















Description
Related PRS (if any):
This frontend PR is related to the backend development branch.
Main changes explained:
How to test:
npm installandnpm startto run this PR locallyScreenshots or videos of changes:
Note: