Avoid nodesByPos corruption during LayoutGroup.restorePos() due to ordering

2019-07-05 05:50:03 +00:00
parent e26463047a
commit 37002541d6
1 changed files with 5 additions and 0 deletions
--- a/Layout.js
+++ b/Layout.js
@@ -97,10 +97,15 @@ class Layout {
          continue;
        }
        obj.savePos();
+        let nodesByPos = new Map(this.nodesByPos_);
        obj.moveBy(offset);
        this.setTension(objects);
        let testTension = this.getTotalTension(objects);
        obj.restorePos();
+        this.nodesByPos_.clear();
+        for (let [k, v] of nodesByPos) {
+          this.nodesByPos_.set(k, v);
+        }
        if (testTension < newTension) {
          newOffset = offset;
          newTension = testTension;