SVG importer does not apply root viewBox scaling to stroke widths

[Oddball777]

When the document width/height is different from the viewBox, geometry is scaled accordingly but not stroke widths (even when the aspect ratio is the same for both). Here's a minimal working (or in this case, failing?) example:

<svg xmlns="http://www.w3.org/2000/svg" width="240" height="120" viewBox="0 0 960 480">
  <line x1="120" y1="140" x2="840" y2="140"
        stroke="#d62828" stroke-width="80" stroke-linecap="butt"/>
  <rect x="120" y="300" width="720" height="80" fill="#1565c0"/>
</svg>

These two objects should have the same visual thickness: one is a line with stroke width of 80, another is a rectangle with height 80, no stroke.
Opening this file directly in Safari shows the intended result (also reproduced with macOS Preview, Inkscape, Firefox and Chrome):

But importing this in Graphite gives this:

I reproduced this issue on:

• RC6 macOS desktop app
• Graphite web app
• https://dev.graphite.art with the following About this release info:

Release Series: Alpha 5
Branch: master
Commit: e725f550
2026-09-07 02:57 Eastern Time

The issue is that the viewport is 240 × 120 while the viewBox is 960 × 480, giving a scale factor of 0.25. Both the 80-unit stroke width and the 80-unit rectangle height should render as 20 units/pixels thick. A workaround for now is simply to use the same values for width/height as for the viewBox.

Possibly related: PR #3942, issue #2435.

[0HyperCube]

One thing I noticed was that the stroke node should be before the transform node. Another thing is that the artboard is often the wrong size with the viewBox since it is already applied in an SVG. Sample fix:

diff --git a/editor/src/messages/portfolio/document/graph_operation/utility_types.rs b/editor/src/messages/portfolio/document/graph_operation/utility_types.rs
index 3149b1d717..3b4a293c91 100644
--- a/editor/src/messages/portfolio/document/graph_operation/utility_types.rs
+++ b/editor/src/messages/portfolio/document/graph_operation/utility_types.rs
@@ -157,15 +157,6 @@ impl<'a> ModifyInputsContext<'a> {
 		self.network_interface.insert_node(shape_id, shape, &[]);
 		self.network_interface.move_node_to_chain_start(&shape_id, layer, &[], self.import);
 
-		if include_transform {
-			let transform = resolve_proto_node_type(graphene_std::transform_nodes::transform::IDENTIFIER)
-				.expect("Transform node does not exist")
-				.default_node_template();
-			let transform_id = NodeId::new();
-			self.network_interface.insert_node(transform_id, transform, &[]);
-			self.network_interface.move_node_to_chain_start(&transform_id, layer, &[], self.import);
-		}
-
 		if include_stroke {
 			let stroke = resolve_proto_node_type(graphene_std::vector_nodes::stroke::IDENTIFIER)
 				.expect("Stroke node does not exist")
@@ -183,6 +174,16 @@ impl<'a> ModifyInputsContext<'a> {
 			self.network_interface.insert_node(fill_id, fill, &[]);
 			self.network_interface.move_node_to_chain_start(&fill_id, layer, &[], self.import);
 		}
+
+		// Ensure the transform node occurs after the stroke node.
+		if include_transform {
+			let transform = resolve_proto_node_type(graphene_std::transform_nodes::transform::IDENTIFIER)
+				.expect("Transform node does not exist")
+				.default_node_template();
+			let transform_id = NodeId::new();
+			self.network_interface.insert_node(transform_id, transform, &[]);
+			self.network_interface.move_node_to_chain_start(&transform_id, layer, &[], self.import);
+		}
 	}
 
 	pub fn insert_text(&mut self, text: String, font: Font, typesetting: TypesettingConfig, layer: LayerNodeIdentifier) {
diff --git a/editor/src/messages/portfolio/portfolio_message_handler.rs b/editor/src/messages/portfolio/portfolio_message_handler.rs
index 847662e4f0..e06dddf1d7 100644
--- a/editor/src/messages/portfolio/portfolio_message_handler.rs
+++ b/editor/src/messages/portfolio/portfolio_message_handler.rs
@@ -1026,33 +1026,11 @@ impl MessageHandler<PortfolioMessage, PortfolioMessageContext<'_>> for Portfolio
 					name: name.clone().unwrap_or_default(),
 				});
 
-				// Parse the SVG to extract its declared canvas origin and dimensions from the viewBox attribute.
-				// This preserves the full canvas rather than measuring only the tighter rendered content bounding box.
-				let artboard_canvas = usvg::roxmltree::Document::parse(&svg)
-					.ok()
-					.and_then(|doc| {
-						let vb = doc.root_element().attribute("viewBox")?;
-						let nums: Vec<f64> = vb
-							.split(|c: char| c.is_ascii_whitespace() || c == ',')
-							.filter(|s| !s.is_empty())
-							.filter_map(|s| s.parse().ok())
-							.collect();
-						if nums.len() >= 4 {
-							Some((
-								glam::IVec2::new(nums[0].round() as i32, nums[1].round() as i32),
-								glam::IVec2::new(nums[2].round() as i32, nums[3].round() as i32),
-							))
-						} else {
-							None
-						}
-					})
-					.or_else(|| {
-						// Fall back to the viewport size when there is no viewBox attribute
-						usvg::Tree::from_str(&svg, &usvg::Options::default()).ok().map(|tree| {
-							let size = tree.size();
-							(glam::IVec2::ZERO, glam::IVec2::new(size.width().round() as i32, size.height().round() as i32))
-						})
-					});
+				// Use the viewport size (the viewbox is already applied by the SVG importer so can be ignored)
+				let artboard_canvas = usvg::Tree::from_str(&svg, &usvg::Options::default()).ok().map(|tree| {
+					let size = tree.size();
+					(glam::IVec2::ZERO, glam::IVec2::new(size.width().round() as i32, size.height().round() as i32))
+				});
 
 				responses.add(DocumentMessage::InsertSvg {
 					name,