@html-next/vertical-collection 4.0.2 → 5.0.0

package/{addon → src}/-private/data-view/skip-list.js

@@ -1,7 +1,7 @@
 import { assert } from '@ember/debug';
 import { DEBUG } from '@glimmer/env';
 
-import roundTo from './utils/round-to';
+import roundTo from './utils/round-to.js';
 
 /*
  * `SkipList` is a data structure designed with two main uses in mind:
@@ -83,7 +83,7 @@ export default class SkipList {
         fill(layer, defaultValue);
 
         left = prevLayer[(length - 1) * 2] || 0;
-        right = prevLayer[((length - 1) * 2) + 1] || 0;
+        right = prevLayer[(length - 1) * 2 + 1] || 0;
 
         // Layers are not powers of 2, and sometimes they may by odd sizes.
         // Only the last value of a layer will be different, so we calculate
@@ -92,7 +92,7 @@ export default class SkipList {
       } else {
         for (i = 0; i < length; i++) {
           left = prevLayer[i * 2];
-          right = prevLayer[(i * 2) + 1];
+          right = prevLayer[i * 2 + 1];
           layer[i] = right ? left + right : left;
         }
       }
@@ -101,7 +101,12 @@ export default class SkipList {
       prevLayer = layer;
     }
 
-    this.total = layer.length > 0 ? layer.length > 1 ? layer[0] + layer[1] : layer[0] : 0;
+    this.total =
+      layer.length > 0
+        ? layer.length > 1
+          ? layer[0] + layer[1]
+          : layer[0]
+        : 0;
 
     assert('total must be a number', typeof this.total === 'number');
 
@@ -147,7 +152,10 @@ export default class SkipList {
     index = index / 2;
 
     assert('index must be a number', typeof index === 'number');
-    assert('index must be within bounds', index >= 0 && index < this.values.length);
+    assert(
+      'index must be within bounds',
+      index >= 0 && index < this.values.length,
+    );
 
     totalAfter = total - (totalBefore + values[index]);
 
@@ -190,7 +198,10 @@ export default class SkipList {
     assert('value must be a number', typeof value === 'number');
     assert('value must non-negative', value >= 0);
     assert('index must be a number', typeof index === 'number');
-    assert('index must be within bounds', index >= 0 && index < this.values.length);
+    assert(
+      'index must be within bounds',
+      index >= 0 && index < this.values.length,
+    );
 
     const { layers } = this;
     const oldValue = layers[layers.length - 1][index];
@@ -216,11 +227,7 @@ export default class SkipList {
   }
 
   prepend(numPrepended) {
-    const {
-      values: oldValues,
-      length: oldLength,
-      defaultValue
-    } = this;
+    const { values: oldValues, length: oldLength, defaultValue } = this;
 
     const newLength = numPrepended + oldLength;
 
@@ -234,11 +241,7 @@ export default class SkipList {
   }
 
   append(numAppended) {
-    const {
-      values: oldValues,
-      length: oldLength,
-      defaultValue
-    } = this;
+    const { values: oldValues, length: oldLength, defaultValue } = this;
 
     const newLength = numAppended + oldLength;
 
@@ -252,11 +255,7 @@ export default class SkipList {
   }
 
   reset(newLength) {
-    const {
-      values: oldValues,
-      length: oldLength,
-      defaultValue
-    } = this;
+    const { values: oldValues, length: oldLength, defaultValue } = this;
 
     if (oldLength === newLength) {
       return;

package/{addon → src}/-private/data-view/utils/insert-range-before.js

@@ -1,4 +1,9 @@
-export default function insertRangeBefore(parent, element, firstNode, lastNode) {
+export default function insertRangeBefore(
+  parent,
+  element,
+  firstNode,
+  lastNode,
+) {
   let nextNode;
 
   while (firstNode) {

package/{addon → src}/-private/data-view/utils/mutation-checkers.js

@@ -1,6 +1,6 @@
 import { get } from '@ember/object';
-import objectAt from './object-at';
-import keyForItem from '../../ember-internals/key-for-item';
+import objectAt from './object-at.js';
+import keyForItem from '../../ember-internals/key-for-item.js';
 
 export function isPrepend(lenDiff, newItems, key, oldFirstKey, oldLastKey) {
   const newItemsLength = get(newItems, 'length');
@@ -10,7 +10,11 @@ export function isPrepend(lenDiff, newItems, key, oldFirstKey, oldLastKey) {
   }
 
   const newFirstKey = keyForItem(objectAt(newItems, lenDiff), key, lenDiff);
-  const newLastKey = keyForItem(objectAt(newItems, newItemsLength - 1), key, newItemsLength - 1);
+  const newLastKey = keyForItem(
+    objectAt(newItems, newItemsLength - 1),
+    key,
+    newItemsLength - 1,
+  );
 
   return oldFirstKey === newFirstKey && oldLastKey === newLastKey;
 }
@@ -23,7 +27,11 @@ export function isAppend(lenDiff, newItems, key, oldFirstKey, oldLastKey) {
   }
 
   const newFirstKey = keyForItem(objectAt(newItems, 0), key, 0);
-  const newLastKey = keyForItem(objectAt(newItems, newItemsLength - lenDiff - 1), key, newItemsLength - lenDiff - 1);
+  const newLastKey = keyForItem(
+    objectAt(newItems, newItemsLength - lenDiff - 1),
+    key,
+    newItemsLength - lenDiff - 1,
+  );
 
   return oldFirstKey === newFirstKey && oldLastKey === newLastKey;
 }

package/src/-private/data-view/utils/object-at.js

@@ -0,0 +1,10 @@
+import { assert } from '@ember/debug';
+
+export default function objectAt(arr, index) {
+  assert(
+    'arr must be an instance of a Javascript Array or implement `objectAt`',
+    Array.isArray(arr) || typeof arr.objectAt === 'function',
+  );
+
+  return arr.objectAt ? arr.objectAt(index) : arr[index];
+}

package/{addon → src}/-private/data-view/utils/scroll-handler.js

@@ -1,6 +1,6 @@
 import { begin, end } from '@ember/runloop';
 import { scheduler } from 'ember-raf-scheduler';
-import SUPPORTS_PASSIVE from './supports-passive';
+import SUPPORTS_PASSIVE from './supports-passive.js';
 const DEFAULT_ARRAY_SIZE = 10;
 const UNDEFINED_VALUE = Object.create(null);
 
@@ -33,11 +33,11 @@ export class ScrollHandler {
       cache = this.handlers[index] = {
         top: element.scrollTop,
         left: element.scrollLeft,
-        handlers
+        handlers,
       };
       // TODO add explicit test
       if (SUPPORTS_PASSIVE) {
-        cache.passiveHandler = function() {
+        cache.passiveHandler = function () {
           ScrollHandler.triggerElementHandlers(element, cache);
         };
       } else {
@@ -50,10 +50,16 @@ export class ScrollHandler {
     }
 
     // TODO add explicit test
-    if (this.isUsingPassive && handlers.length === 1) {
-      element.addEventListener('scroll', cache.passiveHandler, { capture: true, passive: true });
+    if (this.isUsingPassive) {
+      // Only add the event listener once, if more handlers are present.
+      if (handlers.length === 1) {
+        element.addEventListener('scroll', cache.passiveHandler, {
+          capture: true,
+          passive: true,
+        });
+      }
 
-    // TODO add explicit test
+      // TODO add explicit test
     } else if (!this.isPolling) {
       this.poll();
     }
@@ -88,12 +94,16 @@ export class ScrollHandler {
 
         // TODO add explicit test
         if (this.isUsingPassive) {
-          element.removeEventListener('scroll', elementCache.passiveHandler, { capture: true, passive: true });
+          element.removeEventListener('scroll', elementCache.passiveHandler, {
+            capture: true,
+            passive: true,
+          });
         }
       }
-
     } else {
-      throw new Error('Attempted to remove a handler from an unknown element or an element with no handlers');
+      throw new Error(
+        'Attempted to remove a handler from an unknown element or an element with no handlers',
+      );
     }
   }
 

package/{addon → src}/-private/data-view/utils/supports-passive.js

@@ -5,11 +5,11 @@ try {
     get() {
       supportsPassive = true;
       return supportsPassive;
-    }
+    },
   });
 
   window.addEventListener('test', null, opts);
-} catch(e) {
+} catch {
   // do nothing
 }
 

package/{addon/-private/data-view/elements → src/-private/data-view}/viewport-container.js

@@ -13,58 +13,68 @@
  *
  */
 export function ViewportContainer() {
-
   // A bug occurs in Chrome when we reload the browser at a lower
   // scrollTop, window.scrollY becomes stuck on a single value.
   Object.defineProperty(this, 'scrollTop', {
     get() {
-      return document.body.scrollTop
-        || document.documentElement.scrollTop;
+      return document.body.scrollTop || document.documentElement.scrollTop;
     },
     set(v) {
       document.body.scrollTop = document.documentElement.scrollTop = v;
-    }
+    },
   });
 
   Object.defineProperty(this, 'scrollLeft', {
     get() {
-      return window.scrollX
-        || window.pageXOffset
-        || document.body.scrollLeft
-        || document.documentElement.scrollLeft;
+      return (
+        window.scrollX ||
+        window.pageXOffset ||
+        document.body.scrollLeft ||
+        document.documentElement.scrollLeft
+      );
     },
     set(v) {
-      window.scrollX
-        = window.pageXOffset
-        = document.body.scrollLeft
-        = document.documentElement.scrollLeft = v;
-    }
+      window.scrollX =
+        window.pageXOffset =
+        document.body.scrollLeft =
+        document.documentElement.scrollLeft =
+          v;
+    },
   });
 
   Object.defineProperty(this, 'offsetHeight', {
     get() {
       return window.innerHeight;
-    }
+    },
   });
 }
 
-ViewportContainer.prototype.addEventListener = function addEventListener(event, handler, options) {
+ViewportContainer.prototype.addEventListener = function addEventListener(
+  event,
+  handler,
+  options,
+) {
   return window.addEventListener(event, handler, options);
 };
 
-ViewportContainer.prototype.removeEventListener = function addEventListener(event, handler, options) {
+ViewportContainer.prototype.removeEventListener = function addEventListener(
+  event,
+  handler,
+  options,
+) {
   return window.removeEventListener(event, handler, options);
 };
 
-ViewportContainer.prototype.getBoundingClientRect = function getBoundingClientRect() {
-  return {
-    height: window.innerHeight,
-    width: window.innerWidth,
-    top: 0,
-    left: 0,
-    right: window.innerWidth,
-    bottom: window.innerHeight
+ViewportContainer.prototype.getBoundingClientRect =
+  function getBoundingClientRect() {
+    return {
+      height: window.innerHeight,
+      width: window.innerWidth,
+      top: 0,
+      left: 0,
+      right: window.innerWidth,
+      bottom: window.innerHeight,
+    };
   };
-};
 
 export default new ViewportContainer();

package/{addon → src}/-private/ember-internals/key-for-item.js

@@ -1,16 +1,22 @@
 import { get } from '@ember/object';
 import { assert } from '@ember/debug';
 
-import identity from './identity';
+import identity from './identity.js';
 
 export default function keyForItem(item, keyPath, index) {
   let key;
 
-  assert(`keyPath must be a string, received: ${keyPath}`, typeof keyPath === 'string');
+  assert(
+    `keyPath must be a string, received: ${keyPath}`,
+    typeof keyPath === 'string',
+  );
 
   switch (keyPath) {
     case '@index':
-      assert(`A numerical index must be supplied for keyForItem when keyPath is @index, received: ${index}`, typeof index === 'number');
+      assert(
+        `A numerical index must be supplied for keyForItem when keyPath is @index, received: ${index}`,
+        typeof index === 'number',
+      );
       key = index;
       break;
     case '@identity':

package/{addon → src}/-private/index.js

@@ -1,15 +1,15 @@
-export { default as keyForItem } from './ember-internals/key-for-item';
+export { default as keyForItem } from './ember-internals/key-for-item.js';
 
-export { default as closestElement } from './utils/element/closest';
+export { default as closestElement } from './utils/element/closest.js';
 
-export { default as DynamicRadar } from './data-view/radar/dynamic-radar';
-export { default as StaticRadar } from './data-view/radar/static-radar';
+export { default as DynamicRadar } from './data-view/radar/dynamic-radar.js';
+export { default as StaticRadar } from './data-view/radar/static-radar.js';
 
-export { default as ViewportContainer } from './data-view/viewport-container';
-export { default as objectAt } from './data-view/utils/object-at';
+export { default as ViewportContainer } from './data-view/viewport-container.js';
+export { default as objectAt } from './data-view/utils/object-at.js';
 
 export {
   addScrollHandler,
   removeScrollHandler,
-  ScrollHandler
-} from './data-view/utils/scroll-handler';
+  ScrollHandler,
+} from './data-view/utils/scroll-handler.js';

package/{addon → src}/-private/utils/element/closest.js

@@ -1,9 +1,15 @@
-const VENDOR_MATCH_FNS = ['matches', 'webkitMatchesSelector', 'mozMatchesSelector', 'msMatchesSelector', 'oMatchesSelector'];
+const VENDOR_MATCH_FNS = [
+  'matches',
+  'webkitMatchesSelector',
+  'mozMatchesSelector',
+  'msMatchesSelector',
+  'oMatchesSelector',
+];
 let ELEMENT_MATCH_FN;
 
 function setElementMatchFn(el) {
   VENDOR_MATCH_FNS.forEach((fn) => {
-    if ((ELEMENT_MATCH_FN === undefined) && (typeof el[fn] === 'function')) {
+    if (ELEMENT_MATCH_FN === undefined && typeof el[fn] === 'function') {
       ELEMENT_MATCH_FN = fn;
     }
   });

package/{addon → src}/-private/utils/element/estimate-element-height.js

@@ -1,7 +1,10 @@
 import { assert } from '@ember/debug';
 
 export default function estimateElementHeight(element, fallbackHeight) {
-  assert(`You called estimateElement height without a fallbackHeight`, fallbackHeight);
+  assert(
+    `You called estimateElement height without a fallbackHeight`,
+    fallbackHeight,
+  );
   assert(`You called estimateElementHeight without an element`, element);
 
   if (fallbackHeight.indexOf('%') !== -1) {
@@ -21,12 +24,15 @@ function getPercentageHeight(element, fallbackHeight) {
   let parentHeight = element.offsetHeight;
   let percent = parseFloat(fallbackHeight);
 
-  return (percent * parentHeight / 100.0);
+  return (percent * parentHeight) / 100.0;
 }
 
 function getEmHeight(element, fallbackHeight) {
-  const fontSizeElement = fallbackHeight.indexOf('rem') !== -1 ? document.documentElement : element;
-  const fontSize = window.getComputedStyle(fontSizeElement).getPropertyValue('font-size');
+  const fontSizeElement =
+    fallbackHeight.indexOf('rem') !== -1 ? document.documentElement : element;
+  const fontSize = window
+    .getComputedStyle(fontSizeElement)
+    .getPropertyValue('font-size');
 
-  return (parseFloat(fallbackHeight) * parseFloat(fontSize));
+  return parseFloat(fallbackHeight) * parseFloat(fontSize);
 }