Commit 8cc87a7a authored by Sigurd Schneider's avatar Sigurd Schneider Committed by Commit Bot

[turbolizer] Fix graph update bug

D3.v5 needs a merge before applying combined update and
enter actions.

Bug: v8:7327
Change-Id: If5a044c18e4229c5eadb18c18aea191bcbacef32
Reviewed-on: https://chromium-review.googlesource.com/1095188Reviewed-by: 's avatarGeorg Neis <neis@chromium.org>
Commit-Queue: Sigurd Schneider <sigurds@chromium.org>
Cr-Commit-Position: refs/heads/master@{#53632}
parent 9ce6e39e
......@@ -840,11 +840,13 @@ class GraphView extends View implements PhaseView {
}
});
selNodes.select<SVGTextElement>('.type').each(function (d) {
const newAndOldNodes = newGs.merge(selNodes);
newAndOldNodes.select<SVGTextElement>('.type').each(function (d) {
this.setAttribute('visibility', graph.state.showTypes ? 'visible' : 'hidden');
});
selNodes
newAndOldNodes
.classed("selected", function (n) {
if (state.selection.isSelected(n)) return true;
return false;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment