MD Settings: Save and restore scroll position in iron-list

ui/webui/resources/cr_elements/cr_scrollable_behavior.js

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview Behavior for scrollable containers with <iron-list>.
  *
  * Any containers with the 'scrollable' attribute set will have the following
  * classes toggled appropriately: can-scroll, is-scrolled, scrolled-to-bottom.
  * These classes are used to style the container div and list elements
@@ skipping 18 matching lines @@
  *   </template>
  * </dom-module>
  *
  * NOTE: If 'container' is not fixed size, it is important to call
  * updateScrollableContents() when [[items]] changes, otherwise the container
  * will not be sized correctly.
  */
 
 /** @polymerBehavior */
 var CrScrollableBehavior = {
-  properties: {
-    /** @private {number|null} */
-    intervalId_: {type: Number, value: null}
-  },
+
+  /** @private {number|null} */
+  intervalId_: null,
+
+  /**
+   * A dictionary of integer arrays, one entry per scrollable contianer id.
+   * Each array keeps a queue of scroll positions. This ensures that multiple
+   * calls to save/restore scroll do not interfere with each other.
+   * @private {!Object<!Array<number>>}
+   */
+  saveScrollTop_: {},
 
   ready: function() {
     var scrollableElements = this.root.querySelectorAll('[scrollable]');
 
     // Setup the intial scrolling related classes for each scrollable container.
     requestAnimationFrame(function() {
       for (var i = 0; i < scrollableElements.length; i++)
         this.updateScroll_(scrollableElements[i]);
     }.bind(this));
 
@@ skipping 33 matching lines @@
       }
       if (unreadyNodes.length == 0) {
         window.clearInterval(this.intervalId_);
         this.intervalId_ = null;
       } else {
         nodeList = unreadyNodes;
       }
     }.bind(this), 10);
   },
 
+  /** @param {!IronListElement} list */
+  saveScroll(list) {

[Dan Beam, 2017/01/31 04:06:44]

drop ES6 stuff

[stevenjb, 2017/01/31 18:43:09]

Done.

+    let target = list.scrollTarget;
+    // Ignore scrollTop of 0 in case it is intermittent (we do not need to
+    // explicity scroll to 0).
+    let scroll = target.scrollTop || -1;

[Dan Beam, 2017/01/31 04:06:44]

why wouldn't we just do

list.prevScrollTop_ = list.scrollTarget.scrollTop

[stevenjb, 2017/01/31 18:43:09]

Code evolution. Done.

+    this.saveScrollTop_[target.id] = this.saveScrollTop_[target.id] || [];
+    this.saveScrollTop_[target.id].push(scroll);
+  },
+
+  /** @param {!IronListElement} list */
+  restoreScroll(list) {
+    let target = list.scrollTarget;
+    this.async(function() {
+      let scroll = this.saveScrollTop_[target.id].shift();
+      if (scroll != -1)
+        list.scroll(0, scroll);

[Dan Beam, 2017/01/31 04:06:44]

if (list.prevScrollTop_)
  list.scroll(0, list.prevScrollTop_);

[stevenjb, 2017/01/31 18:43:09]

Done.

+    });
+  },
+
   /**
    * Event wrapper for updateScroll_.
    * @param {!Event} event
    * @private
    */
   updateScrollEvent_: function(event) {
     var scrollable = /** @type {!HTMLElement} */ (event.target);
     this.updateScroll_(scrollable);
   },
 
   /**
    * This gets called once intially and any time a scrollable container scrolls.
    * @param {!HTMLElement} scrollable
    * @private
    */
   updateScroll_: function(scrollable) {
     scrollable.classList.toggle(
         'can-scroll', scrollable.clientHeight < scrollable.scrollHeight);
     scrollable.classList.toggle('is-scrolled', scrollable.scrollTop > 0);
     scrollable.classList.toggle(
         'scrolled-to-bottom', scrollable.scrollTop + scrollable.clientHeight >=
             scrollable.scrollHeight);
   },
 };