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 dynamic images in iOS13 #1630

Merged
merged 1 commit into from
Aug 23, 2019

Conversation

vovasty
Copy link
Contributor

@vovasty vovasty commented Aug 21, 2019

iOS13 introduced image variants for dark and light modes. This change provides support for such kind of images.

NB
shall we have a separate cache for dark/light images?

@CLAassistant
Copy link

CLAassistant commented Aug 21, 2019

CLA assistant check
All committers have signed the CLA.

@vovasty vovasty force-pushed the dynamic-images branch 2 times, most recently from 186f52e to 868448c Compare August 21, 2019 20:06
@nguyenhuy nguyenhuy assigned ghost and unassigned ghost Aug 21, 2019
@nguyenhuy
Copy link
Member

nguyenhuy commented Aug 22, 2019

@vovasty Please rebase with latest master to get the right CI configs. Thanks.

@vovasty vovasty force-pushed the dynamic-images branch 4 times, most recently from 12db9df to c12c04c Compare August 23, 2019 18:08
Copy link
Contributor

@bolsinga bolsinga left a comment

Choose a reason for hiding this comment

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

Thanks for the tests!

@vovasty vovasty requested a review from bolsinga August 23, 2019 21:22
Copy link
Contributor

@rahul-malik rahul-malik left a comment

Choose a reason for hiding this comment

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

Looks good, will merge when complete

@rahul-malik rahul-malik merged commit 1961a5a into TextureGroup:master Aug 23, 2019
matthewd1234 pushed a commit to matthewd1234/Texture that referenced this pull request Aug 25, 2019
…eGroup#1630)

Currently ASImageNode would initially load the correct asset from the asset catalog but would not respond to changes in trait collections that would require loading a different asset from the asset catalog. This change allows use to use `-[UIImage imageAsset]` to reference the asset catalog entry for the current image and load the appropriate version based on the current trait collection
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants