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

Add an option to scale down large images on iOS #787

Merged
merged 5 commits into from Oct 6, 2016

Conversation

gsempe
Copy link
Contributor

@gsempe gsempe commented Jun 30, 2014

Difference with the previous PR #769:

  • I have updated the scale down code, following the sample code of Apple "Large Image Downsizing". It is now perfectly safe to download large images with SDWebImage. Note: if the image is very large it can take time several seconds to get the image

Comment from the previous PR:

  • I added an option to scale down large images as it is a common problem with SDWebImage.
  • I chose to not add another method to SDWebImageManagerDelegate because I think we could choose the right maximum size for the library users.
    Let me know what you think?

PS: I create a new pull request because I think my previous one closed can't be updated

@bpoplauschi bpoplauschi modified the milestones: 3.7.0, Future Jul 2, 2014
@bpoplauschi bpoplauschi modified the milestones: 4.0.0, Future Nov 5, 2014
@bpoplauschi bpoplauschi merged commit 1fdaa4e into SDWebImage:master Oct 6, 2016
bpoplauschi added a commit that referenced this pull request Oct 6, 2016
Add an option to scale down large images on iOS
bpoplauschi added a commit that referenced this pull request Oct 6, 2016
intmain added a commit to intmain/SDWebImage that referenced this pull request Jan 4, 2017
* 'master' of https://github.com/rs/SDWebImage: (196 commits)
  Word corrections
  Fixed SDWebImage#1693 Same URLs didn't cancel download
  Updated links for documentation
  1st draft for the 4.0 Migration Guide. Described the changes per component. Now we need to add examples for the most common cases.
  Marked the `SDWebImageManager` `initWithCache:downloader:` as designated initializer.
  Updated diagrams according to current 4.0.0 status
  Updated CHANGELOG (date)
  Bumped version to 4.0.0-beta2
  Updated CHANGELOG
  Fixed SDWebImage#1668 CGContextDrawImage: invalid context 0x0
  Updated CHANGELOG
  Added tests for the scale down decoding + test image and updated a bit the other decoder tests
  Test xcodeproj updated to Xcode 8
  Fixed a bug introduced by 00bf467
  No more carthage, d6fa29c removed it, now also remove installing it via brew
  Update for SDWebImage#787 Add an option to scale down large images on iOS - code cleanup
  Changelog update
  Updated the README backwards compatibility section and linked the 4.0 Migration Guide
  Added a draft for the Migration Guide document.
  Added the Docs folder to the Xcode workspace
  ...

# Conflicts:
#	SDWebImage.xcodeproj/project.pbxproj
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.

None yet

2 participants