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 primary and secondary colors #10

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

aonez
Copy link

@aonez aonez commented Nov 5, 2024

Forcing .black and .white does not fit the Dark mode.

Forcing `.black` and `.white` does not fit the Dark mode.
@Metamogul
Copy link
Owner

Hey @aonez, thanks for the contribution! I've tested your addition and it seems like there's still a few elements where text colors are still not compatible with dark mode. You'll find them when quickly searching through the project for Color. Specifically two occurrences of Color.black make that text really hard to read in dark mode. Would you mind changing these too so I can happily merge your MR?

@aonez
Copy link
Author

aonez commented Nov 5, 2024

@Metamogul I did a search for Color before the PR and did not found more places other that here:

.foregroundColor(configuration.isPressed ? Color.white : Color.pink)
.background(configuration.isPressed ? Color.pink : Color.pink.opacity(0.2))

But since it is for a custom button with pink I did not changed that, it looks ok (pink is questionable for standard colors). Can you point me where should be those other occurences?

@Metamogul
Copy link
Owner

@aonez Sure, no problem. Use Cmd+Shift+F (or whatever you mapped the project-wide search to) to open the search menu, and search for color. You'll find numerous uses of Color.black or Color.white instead of Color.primary or Color.secondary, e.g. in the VMSectionView.swift. Using the preview function of Xcode to inspect the result for this specific view and cycling in between dark and bright mode you'll see how the text is nicely readable in bright mode, but black on dark grey doesn't work so well for the text in dark mode. There's more examples for hard to read text when using dark mode. Let me know if this helps you fixing the remaining issues with colors and dark mode.

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.

2 participants