chore: fix prettyPrint option for SVG class

Author: cyberalien

@iconify/tools/package.json

@@ -3,7 +3,7 @@
 	"type": "module",
 	"description": "Collection of functions for cleaning up and parsing SVG for Iconify project",
 	"author": "Vjacheslav Trushkin",
-	"version": "5.0.1",
+	"version": "5.0.2",
 	"license": "MIT",
 	"bugs": "https://github.com/iconify/tools/issues",
 	"homepage": "https://github.com/iconify/tools",

@iconify/tools/src/svg/index.ts

@@ -1,5 +1,5 @@
 import type { IconifyIcon } from '@iconify/types';
-import { iconToSVG, trimSVG } from '@iconify/utils';
+import { iconToSVG, prettifySVG, trimSVG } from '@iconify/utils';
 import type { CommonIconProps } from '../icon-set/types';
 import type { IconifyIconCustomisations } from '@iconify/utils/lib/customisations/defaults';
 import {
@@ -54,7 +54,10 @@ export class SVG {
 				svgAttributes += ' ' + key + '="' + value + '"';
 			}
 
-			return '<svg' + svgAttributes + '>' + data.body + '</svg>';
+			const svg = '<svg' + svgAttributes + '>' + data.body + '</svg>';
+			return prettyPrint === true
+				? prettifySVG(svg) || svg
+				: trimSVG(svg);
 		}
 
 		// Get icon as is if customisations are not set
@@ -75,17 +78,19 @@ export class SVG {
 			}
 		}
 
-		return stringifyXMLContent([$root], {
+		const result = stringifyXMLContent([$root], {
 			prettyPrint,
 		})!;
+
+		// Additional trim for content (such as multi-line "d" attributes) because XML parser does not trim attributes
+		return prettyPrint === false ? trimSVG(result) : result;
 	}
 
 	/**
 	 * Get SVG as string without whitespaces
 	 */
 	toMinifiedString(customisations?: IconifyIconCustomisations): string {
-		// Additional trimming for better result
-		return trimSVG(this.toString(customisations, false));
+		return this.toString(customisations, false);
 	}
 
 	/**

@iconify/tools/tests/svg/svg-test.ts

@@ -118,4 +118,20 @@ describe('Loading SVG', () => {
 		}
 		expect(threwError).toBe(true);
 	});
+
+	test('Bugged SVG', () => {
+		const svg = new SVG(
+			'<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24"><g fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M12 3h7v18h-14v-18h7Z"/><path d="M14.5 3.5v3h-5v-3"/><path d="M9 13l2 2l4 -4" stroke-dasharray="10" stroke-dashoffset="10"><animate attributeName="stroke-dashoffset" values="10;0" dur="0.2s" fill="freeze"/></path></g><path d="M6 4H10V6H14V4H18V20H6V4Z" fill="currentColor" fill-opacity="0.3"/></svg>'
+		);
+		expect(
+			svg.toString(
+				{
+					height: 'auto',
+				},
+				true
+			)
+		).toBe(
+			'<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24" viewBox="0 0 24 24">\n\t<g fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round">\n\t\t<path d="M12 3h7v18h-14v-18h7Z" />\n\t\t<path d="M14.5 3.5v3h-5v-3" />\n\t\t<path d="M9 13l2 2l4 -4" stroke-dasharray="10" stroke-dashoffset="10">\n\t\t\t<animate attributeName="stroke-dashoffset" values="10;0" dur="0.2s" fill="freeze" />\n\t\t</path>\n\t</g>\n\t<path d="M6 4H10V6H14V4H18V20H6V4Z" fill="currentColor" fill-opacity="0.3" />\n</svg>\n'
+		);
+	});
 });