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

Dark mode toggle #112

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

aaditx23
Copy link

@aaditx23 aaditx23 commented Dec 2, 2024

Added dark mode toggle in settings

@aaditx23 aaditx23 marked this pull request as draft December 2, 2024 13:15
@aaditx23 aaditx23 closed this Dec 2, 2024
@aaditx23 aaditx23 reopened this Dec 2, 2024
@aaditx23 aaditx23 marked this pull request as ready for review December 2, 2024 14:03
@aniruddha-adhikary
Copy link
Owner

@aaditx23 Can you please add screenshots and video recording of it working please?

@aaditx23
Copy link
Author

aaditx23 commented Dec 2, 2024

Yes. Here is the preview

2024-12-02.21-36-24.mp4

@aaditx23
Copy link
Author

aaditx23 commented Dec 2, 2024

mrt_dark
mrt_light_bn
mrt_light

@Mahmud0808
Copy link
Contributor

It seems there's an issue with the status bar color.

@aaditx23
Copy link
Author

aaditx23 commented Dec 2, 2024

@Mahmud0808 the issue you are talking about, is this it?

Screenshot_20241202_221904

@Mahmud0808
Copy link
Contributor

@Mahmud0808 the issue you are talking about, is this it?

Screenshot_20241202_221904

Yes, statusbar content doesn't follow the theme selected. Instead it follows the system theme.

@aaditx23
Copy link
Author

aaditx23 commented Dec 2, 2024

@Mahmud0808 the issue you are talking about, is this it?
Screenshot_20241202_221904

Yes, statusbar content doesn't follow the theme selected. Instead it follows the system theme.

Fixing it...

@aaditx23
Copy link
Author

aaditx23 commented Dec 2, 2024

@Mahmud0808 the issue you are talking about, is this it?
Screenshot_20241202_221904

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.
sta-d
sta-l

EDIT: Issue fixed in android, looking for a mac to work on the ios variant

@aaditx23
Copy link
Author

aaditx23 commented Dec 9, 2024

The issue is fixed for both android and ios.
Here is a screen record:

2024-12-09.20-26-17.mp4

@aniruddha-adhikary
Copy link
Owner

Will review this tonight!

@aniruddha-adhikary
Copy link
Owner

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?

@aaditx23
Copy link
Author

aaditx23 commented Dec 10, 2024

@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.

@aaditx23
Copy link
Author

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

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