From f1a87bab10e0807e2fe8935857bb1fe39bffb80b Mon Sep 17 00:00:00 2001
From: Tim van der Meij <timvandermeij@gmail.com>
Date: Sun, 25 Jun 2017 22:28:46 +0200
Subject: [PATCH] SVG: move setting the stroke properties to the `stroke`
 method

In general, we may not know the stroke properties when path construction
happens. Since we must know the properties when we apply the stroke, we
should set the properties at that point. Note that we already do that
for the color and opacity, but not yet for the other properties.
---
 src/display/svg.js | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/src/display/svg.js b/src/display/svg.js
index 074fce69d..bff93abbe 100644
--- a/src/display/svg.js
+++ b/src/display/svg.js
@@ -882,16 +882,6 @@ SVGGraphics = (function SVGGraphicsClosure() {
         }
       }
       current.path.setAttributeNS(null, 'd', d.join(' '));
-      current.path.setAttributeNS(null, 'stroke-miterlimit',
-                                  pf(current.miterLimit));
-      current.path.setAttributeNS(null, 'stroke-linecap', current.lineCap);
-      current.path.setAttributeNS(null, 'stroke-linejoin', current.lineJoin);
-      current.path.setAttributeNS(null, 'stroke-width',
-                                  pf(current.lineWidth) + 'px');
-      current.path.setAttributeNS(null, 'stroke-dasharray',
-                                  current.dashArray.map(pf).join(' '));
-      current.path.setAttributeNS(null, 'stroke-dashoffset',
-                                  pf(current.dashPhase) + 'px');
       current.path.setAttributeNS(null, 'fill', 'none');
 
       this._ensureTransformGroup().appendChild(current.path);
@@ -1005,9 +995,21 @@ SVGGraphics = (function SVGGraphicsClosure() {
 
     stroke: function SVGGraphics_stroke() {
       var current = this.current;
+
       current.element.setAttributeNS(null, 'stroke', current.strokeColor);
       current.element.setAttributeNS(null, 'stroke-opacity',
                                      current.strokeAlpha);
+      current.element.setAttributeNS(null, 'stroke-miterlimit',
+                                     pf(current.miterLimit));
+      current.element.setAttributeNS(null, 'stroke-linecap', current.lineCap);
+      current.element.setAttributeNS(null, 'stroke-linejoin', current.lineJoin);
+      current.element.setAttributeNS(null, 'stroke-width',
+                                     pf(current.lineWidth) + 'px');
+      current.element.setAttributeNS(null, 'stroke-dasharray',
+                                     current.dashArray.map(pf).join(' '));
+      current.element.setAttributeNS(null, 'stroke-dashoffset',
+                                     pf(current.dashPhase) + 'px');
+
       current.element.setAttributeNS(null, 'fill', 'none');
     },