Skip to content

Commit

Permalink
Revert "Potential Safari workaround for incorrect Quadratic paths - u…
Browse files Browse the repository at this point in the history
…sing de Casteljau subdivision (one level) on quadratic bezier curves in Safari, see phetsims/graphing-quadratics#206"

This reverts commit fa6d835.
  • Loading branch information
jonathanolson committed Apr 4, 2024
1 parent 8cb5024 commit c8a09e4
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions js/display/drawables/PathSVGDrawable.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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
Expand Down

0 comments on commit c8a09e4

Please sign in to comment.