Skip to content

Commit

Permalink
fixed highlighting issue within pie charts
Browse files Browse the repository at this point in the history
  • Loading branch information
lisastaehli committed Jun 19, 2017
1 parent 7fe64f9 commit 9311430
Showing 1 changed file with 6 additions and 13 deletions.
19 changes: 6 additions & 13 deletions app/support/chartMaker.js
Original file line number Diff line number Diff line change
Expand Up @@ -123,24 +123,17 @@ define([

if (dataItem.pulled) {
chart.pullSlice(dataItem, 0);
if (state === "city" && settings.layer1.visible === true) {
settings.layer2.renderer = applyRenderer.createRenderer(settings.values, settings.color, settings.usagename);
}
if (state === "building" && settings.layer2.visible === true) {
settings.layer1.renderer = applyRenderer.createRenderer(settings.values, settings.color, settings.usagename);
}

settings.layer1.renderer = applyRenderer.createRenderer(settings.values, settings.color, settings.usagename);

view.environment.lighting.directShadowsEnabled = true;
view.environment.lighting.ambientOcclusionEnabled = true;

} else {
chart.pullSlice(dataItem, 1);

if (state === "city" && settings.layer1.visible === true) {
settings.layer2.renderer = applyRenderer.createRenderer(selectedvalues, selectedcolor, settings.usagename);
}
if (state === "building" && settings.layer2.visible) {
settings.layer1.renderer = applyRenderer.createRenderer(selectedvalues, selectedcolor, settings.usagename);
}
settings.layer1.renderer = applyRenderer.createRenderer(selectedvalues, selectedcolor, settings.usagename);

view.environment.lighting.directShadowsEnabled = false;
view.environment.lighting.ambientOcclusionEnabled = false;
}
Expand Down Expand Up @@ -177,4 +170,4 @@ define([
return hex.length == 1 ? "0" + hex : hex;
}
};
});
});

0 comments on commit 9311430

Please sign in to comment.