Merge pull request #20 from rcancro/main

[PATCH] [Bug] update makeTransformation to consider svgs where width

Author: swhitty

Samples/dash.svg

@@ -266,7 +266,7 @@ extension SFSymbolRenderer {
 
     static func makeTransformation(from source: LayerTree.Rect,
                                    to destination: LayerTree.Rect) -> LayerTree.Transform.Matrix {
-        let scale = destination.height / source.height
+        let scale = min(destination.width / source.width, destination.height / source.height)
         let scaleMidX = source.midX * scale
         let scaleMidY = source.midY * scale
         let tx = destination.midX - scaleMidX

SwiftDraw/Renderer.SFSymbol.swift

@@ -54,6 +54,23 @@ final class RendererSFSymbolTests: XCTestCase {
             template.black.contents.paths.isEmpty
         )
     }
+    
+    func testWidthGreaterThanHeightSymbol() throws {
+        // This svg has a width of 24, but a height of 4. When scaling in SFSymbolRenderer makeTransformation
+        // the height was being used to create scale causing the width to be 440 instead of 108.
+        let url = try Bundle.test.url(forResource: "dash.svg")
+        let template = try SFSymbolTemplate.parse(
+            SFSymbolRenderer.render(fileURL: url)
+        )
+
+        XCTAssertTrue(
+            template.regular.bounds.size.width == 108.0
+        )
+        XCTAssertTrue(
+            template.regular.bounds.size.height == 70.0
+        )
+
+    }
 
     #if canImport(CoreGraphics)
     func testStrokeSymbol() throws {