Add dark theme support for byjus.com by AmrendraTheCoder · Pull Request #14509 · darkreader/darkreader · GitHub
Skip to content

Add dark theme support for byjus.com#14509

Closed
AmrendraTheCoder wants to merge 4 commits into
darkreader:mainfrom
AmrendraTheCoder:fix/byjus-dark-theme
Closed

Add dark theme support for byjus.com#14509
AmrendraTheCoder wants to merge 4 commits into
darkreader:mainfrom
AmrendraTheCoder:fix/byjus-dark-theme

Conversation

@AmrendraTheCoder

Copy link
Copy Markdown
Contributor
  • Invert logos and brand elements for better dark mode visibility
  • Fix search box background to use neutral background color
  • Minimal targeted fixes for essential UI elements

- Invert logos and brand elements for better dark mode visibility
- Fix search box background to use neutral background color
- Minimal targeted fixes for essential UI elements

@Myshor Myshor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Found some issues applying this fix.

Comment thread src/config/dynamic-theme-fixes.config Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This selector if you realy need to use it on some picture must be changed to be less general.

I found it breaks some pics on page I landed when I started check.
obraz

Same little lower on the same page:
obraz

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

alright let me fix it.

- Make image selectors more specific to target only header/nav logos
- Prevent content images from being inappropriately inverted
- Scope img[alt] selectors to header, nav, navbar contexts only

Resolves feedback from PR review about broken content images.
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.

3 participants