Skip to content
This repository has been archived by the owner on Nov 7, 2018. It is now read-only.

Fixed multiple lasso bug #125

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixed multiple lasso bug #125

wants to merge 1 commit into from

Commits on Feb 27, 2015

  1. Fixed multiple lasso bug

    A d3.select("path") is called to select the path drawn in order to get the path definition and create a closed path from it. This does not work if multiple lassos are drawn on the page, since the d3.select("path") will only select the first path. Instead of using d3.select() to find this value, it is actually already stored previously in a variable called "path". So the closed path can be created by just using 
    path + "Z";
    
    Instead of d3.select(path). This version allows you to use multiple lasso's on one page.
    skokenes committed Feb 27, 2015
    Configuration menu
    Copy the full SHA
    14ce47f View commit details
    Browse the repository at this point in the history