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

Use "System Default" theme by default #4710

Closed
xmha97 opened this issue Dec 30, 2022 · 11 comments · Fixed by #4781
Closed

Use "System Default" theme by default #4710

xmha97 opened this issue Dec 30, 2022 · 11 comments · Fixed by #4781

Comments

@xmha97
Copy link

xmha97 commented Dec 30, 2022

Please use the "System Default" theme by default.

Screenshot_2022-12-30-21-07-20-110_de.westnordost.streetcomplete.jpg

@westnordost
Copy link
Member

Actually, I think "Auto" should be removed altogether. Why would you let SC switch between light and dark theme automatically but not the rest of your phone as well? @ENT8R what do you think?

@ENT8R
Copy link
Contributor

ENT8R commented Dec 31, 2022

I think the auto option is still a leftover from the time when a system-wide dark mode was not yet widely supported. Since Android version 10, where a system-wide dark mode was introduced, the percentage of people using this setting should not be too large. However, some users might still use this setting before Android 10, so maybe it is reasonable to keep this option for Android versions before 10 but remove it for more recent versions?
And if you decide not to remove this option, MODE_NIGHT_AUTO should probably be replaced with MODE_NIGHT_AUTO_BATTERY, since the first one is deprecated.

@mnalis
Copy link
Member

mnalis commented Dec 31, 2022

Why would you let SC switch between light and dark theme automatically but not the rest of your phone as well?

Because your phone does not have that functionality? Even on phones that have system-wide dark/light mode like my newer EMUI12-based phone (which used to be Android10, but nowdays I can't even tell as they removed that information from About screen) might not have system time-based schedule for turning dark-mode on/off? My older phone I use for SC I don't think even has a concept of Android-level dark/light modes.

So I would definitely leave Auto option in. I'd also be against randomly removing options from preferences depending on android version, that is the recipe for user confusion, especially if there is not a technical reason why it must be done (if there was such a technical reason, I'd still prefer the option to be greyed out instead of removed, with extra link appearing leading to explanation why it cannot be selected).


As for making System Default the SC default on new installs, I definitely see the reasoning for it, and even somewhat lean towards that idea. ..

However, one should note that doing so would likely reduce discoverability of different themes (i.e. with Auto being the default, the user using the SC in various times of the day will be exposed to different SC themes, so knowing that there are choices, might decide to choose the one they like better. I do know know if Android11 and newer automatically switch system-wide dark/light modes by default? Older ones like mine definitely do not have that capability, so users would not be exposed to different themes, so they'd probably not be aware they exist, even if they dislike the currently displayed theme).

@westnordost
Copy link
Member

Currently, 87% of all StreetComplete users on Google Play use Android 10 and above.

@xmha97
Copy link
Author

xmha97 commented Jan 1, 2023

For Android 10 and earlier:
Set "Auto" theme by default.

For Android 10 and later:
Set "System default" theme by default.

(It's VLC's strategy for choosing the default theme)

Note: An option should not be removed, even in Android 10 and later, the Auto option can be useful.

@mnalis
Copy link
Member

mnalis commented Jan 1, 2023

Currently, 87% of all StreetComplete users on Google Play use Android 10 and above.

And when was feature time-based auto-switching Android light/dark mode implemented? Because I'm not seeing it in my Android 10 (EMUI 12). Maybe that is Android 11 or newer (or is only present in some Android 10 versions?). Sadly I don't have access to anything newer than that Android 10, so can't test.

For Android 10 and earlier: Set "Auto" theme by default.
For Android 10 and later: Set "System default" theme by default.
(It's VLC's strategy for choosing the default theme)

@xmha97 That sounds good to me; but do you know what it does if the Android version is exactly 10 ?

@Helium314
Copy link
Collaborator

And when was feature time-based auto-switching Android light/dark mode implemented

This might be vendor specific... I have working system default theme and system-wide autoswitch on Android 9 (LineageOS 16)

@xmha97
Copy link
Author

xmha97 commented Jan 1, 2023

Do you know what it does if the Android version is exactly 10 ?

Set the "System default" theme by default on Android 10,

because system-wide dark mode has been introduced on Android 10 (although it was beta in older Android)

https://developer.android.com/about/versions/10/highlights#dark_theme

@mnalis
Copy link
Member

mnalis commented Jan 1, 2023

https://developer.android.com/about/versions/10/highlights#dark_theme

Does it change automatically by time of day in Android 10, though? (Which is what SC "Auto" mode does, and is quite convenient default, especially as SC bright theme is hard on eyes in the evenings, and SC dark theme has problems being usable in the bright daylight - e.g. see discussions in streetcomplete/streetcomplete-mapstyle#141).

@HolgerJeromin
Copy link
Contributor

Android 10 does support manual dark mode but not switching to it by time.
(I do switch the mode with the app macrodroid though.)
Android 11 has the timebased dark mode.

@mnalis
Copy link
Member

mnalis commented Jan 2, 2023

Android 11 has the timebased dark mode.

Then perhaps Android 11 (and higher) should default to "System Default" SC theme, and previous versions to "Auto" SC theme?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants