From c8a09e4c62008332de760a8037f1692d0c57e556 Mon Sep 17 00:00:00 2001 From: Jonathan Olson Date: Thu, 4 Apr 2024 15:31:53 -0600 Subject: [PATCH] Revert "Potential Safari workaround for incorrect Quadratic paths - using de Casteljau subdivision (one level) on quadratic bezier curves in Safari, see https://github.com/phetsims/graphing-quadratics/issues/206" This reverts commit fa6d835736e76609c8a7ed4bcb30cf720a85029d. --- js/display/drawables/PathSVGDrawable.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/js/display/drawables/PathSVGDrawable.js b/js/display/drawables/PathSVGDrawable.js index cc1520652..534532df2 100644 --- a/js/display/drawables/PathSVGDrawable.js +++ b/js/display/drawables/PathSVGDrawable.js @@ -13,8 +13,6 @@ import { PathStatefulDrawable, scenery, svgns, SVGSelfDrawable } from '../../imp // TODO: change this based on memory and performance characteristics of the platform https://github.com/phetsims/scenery/issues/1581 const keepSVGPathElements = true; // whether we should pool SVG elements for the SVG rendering states, or whether we should free them when possible for memory -const useSafariQuadraticWorkaround = platform.safari; - class PathSVGDrawable extends PathStatefulDrawable( SVGSelfDrawable ) { /** * @public @@ -42,7 +40,7 @@ class PathSVGDrawable extends PathStatefulDrawable( SVGSelfDrawable ) { const path = this.svgElement; if ( this.dirtyShape ) { - let svgPath = this.node.hasShape() ? ( useSafariQuadraticWorkaround ? this.node._shape.getSVGPathWithSafariWorkaround() : this.node._shape.getSVGPath() ) : ''; + let svgPath = this.node.hasShape() ? this.node._shape.getSVGPath() : ''; // temporary workaround for https://bugs.webkit.org/show_bug.cgi?id=78980 // and http://code.google.com/p/chromium/issues/detail?id=231626 where even removing