Merge pull request #24 from wordpress-mobile/fix/drawing-memory-usage

Drawing: Limit scale of rendered annotationview image
wordpress-mobile · Sep 3, 2020 · 8942cb9 · 8942cb9
2 parents 4e066b1 + 8abe0ed
commit 8942cb9
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/MediaEditor.podspec b/MediaEditor.podspec
@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
   s.name             = 'MediaEditor'
-  s.version          = '1.2.0'
+  s.version          = '1.2.1'
   s.summary          = 'An extensible Media Editor for iOS.'
 
   s.description      = <<-DESC

diff --git a/RELEASE-NOTES.txt b/RELEASE-NOTES.txt
@@ -1,3 +1,7 @@
+1.2.1
+-----
+* Improve memory usage of AnnotationView when rendering drawings
+
 1.2.0
 -----
 * Replace TOCropViewController with CropViewController

diff --git a/Sources/Capabilities/Drawing/MediaEditorAnnotationView.swift b/Sources/Capabilities/Drawing/MediaEditorAnnotationView.swift
@@ -178,7 +178,10 @@ class MediaEditorAnnotationView: UIView {
 
         let canvasViewImage = canvasView.drawing.image(from: canvasView.bounds, scale: UIScreen.main.scale)
 
-        let renderer = UIGraphicsImageRenderer(size: targetSize, format: .default())
+        let format = UIGraphicsImageRendererFormat()
+        format.scale = 1
+
+        let renderer = UIGraphicsImageRenderer(size: targetSize, format: format)
         let renderedImage = renderer.image { context in
             imageViewImage.draw(at: .zero)
             canvasViewImage.draw(in: CGRect(origin: .zero, size: targetSize))