Skip to content

Try to solve the SwiftUI bug of rendering EXIF UIImage in WebImage, as well as vector images #102

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

Merged
merged 6 commits into from
Apr 15, 2020

Conversation

dreampiggy
Copy link
Collaborator

Now we use the Image(decorative:) API to grab the CGImage and orientation instead. For vector image, draw vector image into bitmap for rescue.

This close #101

…ow we use the Image(decorative:) API to grab the CGImage and orientation instead. For vector image, draw vector image into bitmap for rescue
@dreampiggy dreampiggy force-pushed the hack_exif_webimage_rendering_aspect_ratio branch from 7be32e9 to 0a41337 Compare April 15, 2020 04:40
…I think SwfitUI will fix this issue in the future and we'd better use UIImage as much as possible
…port in the future. Currently grab the poster image does not do any performance benefit
… think this should be fixed by SwiftUI team :)
@codecov
Copy link

codecov bot commented Apr 15, 2020

Codecov Report

Merging #102 into master will decrease coverage by 2.75%.
The diff coverage is 53.26%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
- Coverage   75.53%   72.78%   -2.76%     
==========================================
  Files           9        9              
  Lines         748      823      +75     
==========================================
+ Hits          565      599      +34     
- Misses        183      224      +41     
Flag Coverage Δ
#ios 72.78% <53.26%> (-2.76%) ⬇️
Impacted Files Coverage Δ
SDWebImageSwiftUI/Classes/Image.swift 40.90% <40.90%> (ø)
SDWebImageSwiftUI/Classes/WebImage.swift 79.68% <64.58%> (-4.83%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 14a3cce...3982797. Read the comment docs.

@dreampiggy dreampiggy merged commit 10ab2a2 into master Apr 15, 2020
@dreampiggy dreampiggy deleted the hack_exif_webimage_rendering_aspect_ratio branch April 15, 2020 12:07
@dreampiggy dreampiggy added apple bug Apple's bug that need workaround webimage WebImage struct labels Apr 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apple bug Apple's bug that need workaround webimage WebImage struct
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't properly render images with exif data
1 participant