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 color mode attribution for app in dark mode #441

Closed
2 tasks done
andrewtavis opened this issue May 18, 2024 · 5 comments
Closed
2 tasks done

Fix color mode attribution for app in dark mode #441

andrewtavis opened this issue May 18, 2024 · 5 comments
Assignees
Labels
-next release- Included in the next release bug Something isn't working help wanted Extra attention is needed

Comments

@andrewtavis
Copy link
Member

andrewtavis commented May 18, 2024

Terms

Behavior

We currently are assigning colors inconsistently in the application in dark mode.

When we start the app we have the correct colors for the install screen background and the tab bar as seen here:

Dark mode fine

Switching to Settings, the tab bar and menu options now have a more grey color:

Dark mode discolored

On About the tab bar is colored again ok, but the chevron menu options have the wrong color:

Dark mode discolored

And if we switch to the installation from settings we get a more grey tab bar (it's correctly black if we switch from About):

Dark mode discolored

Would be great to get some help with this, and I'd be very happy to support!

Device type

iPhone and iPad

Versions

Current main version

@andrewtavis andrewtavis added bug Something isn't working help wanted Extra attention is needed -next release- Included in the next release labels May 18, 2024
@andrewtavis
Copy link
Member Author

From the sync: on launch it's taking the color from the storyboard, and then within the code it's being set to a different color.

@andrewtavis
Copy link
Member Author

@SaurabhJamadagni, I decided that it made sense for me to work on it as a precursor to working on #351 :) 1998974 sends along all the changes except for the settings screen, so almost done!

@andrewtavis
Copy link
Member Author

Note on this: the discoloration is actually not dark mode based. There's an grey/black with a very low alpha being applied above the cells and the tab bar. So far finding them and settings their colors explicitly is working. I've also set the color of the tab bar explicitly to white and black, but then on the settings page it's still getting the light alpha over it and then on the installation page it persists if the user switches from the settings page (still not happening on About).

@andrewtavis
Copy link
Member Author

Removing translucent from the tab bar properties fixes it 🥳 Last thing for the color mode is to fix the link color attributions. It looks like the light mode and dark mode colors are being applied in an opposite manner 🤔 Maybe this is related to the tab bar colors where the Scribe Blue that's displayed is the opposite one for the color mode 🤔

@andrewtavis
Copy link
Member Author

Closed by the above commits 🥳 Thanks for the chat about this in the sync, @SaurabhJamadagni, and hope it's ok that I stepped in! 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-next release- Included in the next release bug Something isn't working help wanted Extra attention is needed
Projects
Archived in project
Development

No branches or pull requests

2 participants