Skip to content

Feature/darkmode : fix(ui) login blank page, password eye icon, and admin dark mode backgrounds#138

Open
imSubhro wants to merge 7 commits into
rdodiya:gssoc_developfrom
imSubhro:feature/darkmode
Open

Feature/darkmode : fix(ui) login blank page, password eye icon, and admin dark mode backgrounds#138
imSubhro wants to merge 7 commits into
rdodiya:gssoc_developfrom
imSubhro:feature/darkmode

Conversation

@imSubhro
Copy link
Copy Markdown
Contributor

Summary

Fixes three UI issues reported by the project owner.

Changes

  1. Fix blank login page
  • src/index.jsx (real entry point) was missing GoogleOAuthProvider wrapper, causing a runtime crash and blank white page
  • Cleaned up src/main.jsx which had broken loose JSX outside ReactDOM.createRoot
  • Fixed index.html @styles/ links that didn't resolve (Vite aliases don't work in HTML tags)
  1. Fix duplicate eye icon on password field
  • Added CSS to hide the browser's native password reveal icon (Edge/IE ::-ms-reveal) which was appearing alongside the custom white eye icon in
    Login.jsx
  1. Admin pages dark mode — background & typography
  • Added admin-content class to in AdminLayout.jsx as a CSS scope anchor
  • Imported global.css in App.jsx (it was never loaded)
  • Added CSS overrides in global.css to force rgb(31 41 55) background and white/light-gray text on all admin components in dark mode — covers all 49
    admin component files without modifying each one individually

Pages affected

Dashboard, Menu, Orders, UPI, QR Code, Profile, Branches, Website

@rdodiya
Copy link
Copy Markdown
Owner

rdodiya commented May 17, 2026

Hi @imSubhro ,
Please revert the package-lock.json and below are some suggestions

The below UI is looked perfect as there is border line and all text is visible which differ cards from background in dark mode but in light mode

image

Light mode that difference is missing

image

Some text color is white which are invisible due to white bg , please fix this

image

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