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

Support dark mode #199

Merged
merged 6 commits into from
Nov 4, 2019
Merged

Support dark mode #199

merged 6 commits into from
Nov 4, 2019

Conversation

WilsonGramer
Copy link
Contributor

This pull request adds support for iOS 13 dark mode by using adaptive UIColors and swizzling traitCollectionDidChange(_:), and changes the default color from .clouds to the new .skeletonDefault (which is the same as .clouds but with a dark variant). It should resolve both #192 and #197.

I also updated the example app to support dark mode, and added my changes to the English README — I'll need some help translating my changes to the other languages, though. Let me know what you think!

If on iOS 13 or higher, returns a UIColor using the
`init(dynamicProvider:)` initializer so it automatically adjusts to
dark mode. Also switches out the `clouds` default color with the
dynamic `skeletonDefault` color.
Make skeleton views respond to changes in system appearance by
updating the skeleton view whenever the trait collection changes,
so the colors change automatically with dark mode.
@Juanpe
Copy link
Owner

Juanpe commented Oct 17, 2019

Hi @Wilsonator5000, thanks for contributing!

Please, could you update the CHANGELOG file too?

Thanks!

Copy link

@leomodro leomodro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a nice solution. Much needed

@WilsonGramer
Copy link
Contributor Author

@Juanpe OK, I updated the CHANGELOG!

@WilsonGramer
Copy link
Contributor Author

Hey @Juanpe, what's the status on this PR? Thanks 😄

@Juanpe Juanpe merged commit a1e183a into Juanpe:master Nov 4, 2019
@Juanpe
Copy link
Owner

Juanpe commented Nov 4, 2019

Thanks for contributing @Wilsonator5000 :)

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