Skip to content

fix: update crop monitoring navbar color for light mode visibility#1642

Merged
omroy07 merged 1 commit intoomroy07:mainfrom
Sunidhi037:fix/crop-light-mode-navbar
Feb 24, 2026
Merged

fix: update crop monitoring navbar color for light mode visibility#1642
omroy07 merged 1 commit intoomroy07:mainfrom
Sunidhi037:fix/crop-light-mode-navbar

Conversation

@Sunidhi037
Copy link
Copy Markdown
Contributor

Which issue does this PR close?

This PR resolves a UI inconsistency on the Smart Crop Monitoring page (crop.html) where the navigation bar remained black/dark even after the user toggled the platform to Light Mode.

Key Enhancements:

Dynamic Surface Colors: Implemented a CSS override for the [data-theme="light"] attribute, forcing the navbar background to a semi-transparent white (rgba(255, 255, 255, 0.9)).

Accessibility Restoration: Corrected text and navigation link colors to deep charcoal (#1e293b) to ensure a high contrast ratio against the new light background.

Visual Polish: Maintained the glassmorphism depth with a 10px backdrop blur and a refined 1px border that adapts to the active theme.

Zero-Crash Integration: The change is purely CSS-driven, ensuring no interference with the existing monitoring logic or Three.js background animations.

BEFORE:
image

AFTER:
image

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 24, 2026

@Sunidhi037 is attempting to deploy a commit to the Om Roy's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@Sunidhi037
Copy link
Copy Markdown
Contributor Author

@omroy07 please put the label of SWoC26 and difficulty label too.

@omroy07 omroy07 merged commit a284554 into omroy07:main Feb 24, 2026
2 of 5 checks passed
@github-actions
Copy link
Copy Markdown

🎉 Congrats @Sunidhi037 on getting your PR merged! 🙌
Thanks for the contribution — looking forward to more from you 🚀

SatyamPandey-07 added a commit to SatyamPandey-07/AgriTech that referenced this pull request Feb 25, 2026
SatyamPandey-07 added a commit to SatyamPandey-07/AgriTech that referenced this pull request Feb 25, 2026
omroy07 added a commit that referenced this pull request Feb 27, 2026
…escrow-1642

feat: Carbon Trading Escrow System #1642
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UI/UX]: Redesign Navbar for Full Light Theme Compatibility

2 participants