-
Notifications
You must be signed in to change notification settings - Fork 99
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
Dark mode toggle #112
base: master
Are you sure you want to change the base?
Dark mode toggle #112
Conversation
@aaditx23 Can you please add screenshots and video recording of it working please? |
Yes. Here is the preview 2024-12-02.21-36-24.mp4 |
It seems there's an issue with the status bar color. |
@Mahmud0808 the issue you are talking about, is this it? |
Yes, statusbar content doesn't follow the theme selected. Instead it follows the system theme. |
Fixing it... |
Made the statusbar transparent for now, so the font is visible. but will work on it a bit later, don't have anything to test the ios version yet. EDIT: Issue fixed in android, looking for a mac to work on the ios variant |
The issue is fixed for both android and ios. 2024-12-09.20-26-17.mp4 |
Will review this tonight! |
Tested, works perfectly @aaditx23! Thank you so much for all the effort. I absolutely love the spirit of us working together to build a better app. I was thinking from the user perspective, the app behavior has changed a bit. Before the app would automatically go to Dark Mode depending on the system. Should we completely remove that functionality? |
@aniruddha-adhikary thank you so much. As we have a toggle now, i guess we can remove it. What do you think? I can try to implement that as well. should be doable. let me know. |
Not following system theme by default might fall under bad UX so i have fixed it. Here is a sped up screen record. fix_follows_system_theme_changed.mp4 |
Added dark mode toggle in settings