Skip to content

Made About and Contact page theme aware#127

Closed
Lalitya31 wants to merge 4 commits intoNsanjayboruds:mainfrom
Lalitya31:About-Contact
Closed

Made About and Contact page theme aware#127
Lalitya31 wants to merge 4 commits intoNsanjayboruds:mainfrom
Lalitya31:About-Contact

Conversation

@Lalitya31
Copy link
Copy Markdown
Contributor

About page in dark and light theme :
image
image

Contact page :
image
image

Changes are tested locally and work with full efficiency.

Copilot AI review requested due to automatic review settings March 8, 2026 06:37
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR makes the About and Contact pages theme-aware by adding Tailwind CSS dark-mode variants alongside new light-mode color values. Previously, both pages only had hard-coded dark color classes; now they conditionally switch between light (white/gray) and dark (dark navy) palettes based on the dark: variant.

Changes:

  • Added bg-white / text-black / text-gray-{n} light mode values to all UI elements in Contact.jsx using the dark: prefix to preserve dark mode appearance.
  • Applied the same light/dark theming pattern to all five sections of About.jsx.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
frontend/src/pages/Contact.jsx Added light-mode backgrounds, text, and border colors using dark: variants for all containers, form inputs, labels, and buttons.
frontend/src/pages/About.jsx Added light-mode backgrounds, text, and border colors using dark: variants across all five page sections (hero, operational overview, stats, capabilities, core principles).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread frontend/src/pages/Contact.jsx Outdated
Comment thread frontend/src/pages/About.jsx Outdated
@Lalitya31 Lalitya31 closed this by deleting the head repository Mar 10, 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.

2 participants