Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix icon colors in dark mode #1223

Merged
merged 2 commits into from
Jul 10, 2024
Merged

Fix icon colors in dark mode #1223

merged 2 commits into from
Jul 10, 2024

Conversation

zatteo
Copy link
Contributor

@zatteo zatteo commented Jul 2, 2024

### ✨ Features

*

### πŸ› Bug Fixes

* Fix icon colors in dark mode

### πŸ”§ Tech

*

Checklist

Before merging this PR, the following things must have been done if relevant:

  • Tested on iOS
  • Tested on Android
  • Test coverage
  • README and documentation

@zatteo zatteo requested review from acezard and Ldoppea as code owners July 2, 2024 09:11
Copy link
Member

@Ldoppea Ldoppea left a comment

Choose a reason for hiding this comment

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

Seen during a call: need to rewrite commit descriptions

zatteo added 2 commits July 10, 2024 10:43
Password view is not compatible with dark mode but it was
using for the icon animation a color which is since recent commits
responsive to dark mode.

So here we force the icon animation color to white to stay consistent
and one day we may convert totally the password view to dark mode.
@zatteo zatteo force-pushed the fix/icon-colors-dark-mode branch from a857ef2 to 3e0934d Compare July 10, 2024 08:46
@zatteo zatteo merged commit 74b03ac into master Jul 10, 2024
1 check passed
@zatteo zatteo deleted the fix/icon-colors-dark-mode branch July 10, 2024 08:56
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