-
-
Notifications
You must be signed in to change notification settings - Fork 287
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
Onboarding legibility issue in Dark Mode #875
Comments
@teolemon Just curious: are we talking about real people that launched the app for the very first time, or developers in dev mode? I ask that because I wonder if the app starts in light mode anyway:
Because if the app automatically inits in light mode, we could ignore issues regarding onboarding in dark mode. |
Yeah, I also thought about this while coding the onboarding flow, the app
just starts in light mode.
…On Fri, Jan 7, 2022, 08:57 monsieurtanuki ***@***.***> wrote:
@teolemon <https://github.com/teolemon> Just curious: are we talking
about real people that launched the app for the very first time, or
developers in dev mode? I ask that because I wonder if the app starts in
light mode anyway:
- is there a "dark mode for every app" set-up on iPhone?
- does smoothie comply to that set-up?
Because if the app automatically inits in light mode, we could ignore
issues regarding onboarding in dark mode.
—
Reply to this email directly, view it on GitHub
<#875 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABZRHYRLHITJW3K5L7QFMBDUU2MILANCNFSM5LOGQWIA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Yes it starts in light + blue theme mode, the buttons also look bad for example with green theme. The theme shouldn't be a problem, but at some point we will switch to automatically identifying the system dark mode status and then this will also be on in onboarding. |
Partly the same as #1356 which has a PR |
Fixed by #1357 |
@M123-dev I believe we still have the top title legibility |
Yeah thats right @teolemon |
Also the next page button has too less opacity |
Let's open a new issue, possibly |
Tracked in #1386 |
What
Part of
Screenshot
The text was updated successfully, but these errors were encountered: