Skip to content

Commit

Permalink
Limited removeal of brushes to histo element in order to not remove p…
Browse files Browse the repository at this point in the history
…ossible external brushes
  • Loading branch information
santilland committed Mar 30, 2016
1 parent 11e61b5 commit f1ce75f
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
8 changes: 4 additions & 4 deletions app/scripts/scatter.js
Original file line number Diff line number Diff line change
Expand Up @@ -51,10 +51,10 @@ function scatterPlot(args, callback, openinfo, filterset) {

this.showDropDownSelection = defaultFor(args.showDropDownSelection, true);;

d3.selectAll(".point-tooltip").remove();
d3.selectAll(".AV-point-tooltip").remove();

this.tooltip = d3.select("body").append("div")
.attr("class", "point-tooltip");
.attr("class", "AV-point-tooltip");


}
Expand Down Expand Up @@ -880,7 +880,7 @@ scatterPlot.prototype.render = function(){

// Close tooltip if interaction is done anywhere else.
$(document).on("mousedown", function(e){
if (e.target.nodeName!="circle" && e.target.className!="point-tooltip"){
if (e.target.nodeName!="circle" && e.target.className!="AV-point-tooltip"){
if(self.selectedpoint){
self.tooltip.transition()
.duration(100)
Expand Down Expand Up @@ -1074,7 +1074,7 @@ scatterPlot.prototype.parallelsPlot = function parallelsPlot(){
this.parameters = this.headerNames.slice(0);

// Do some cleanup (especially on objects with events)
d3.selectAll(".brush").remove();
d3.select(self.histoEl).selectAll(".brush").remove();
d3.select("#reset_filters").remove();

this.parameters.forEach(function(para) {
Expand Down
Loading

0 comments on commit f1ce75f

Please sign in to comment.