From a9ca764e7e23c0c14949c308a9ce54d4fdb9a361 Mon Sep 17 00:00:00 2001
From: Chocobozzz <me@florianbigard.com>
Date: Thu, 22 Feb 2018 16:41:02 +0100
Subject: Throttle infinite scroller

---
 client/src/app/shared/misc/utils.ts                  | 20 ++++++++++++--------
 .../app/shared/video/infinite-scroller.directive.ts  |  5 ++---
 2 files changed, 14 insertions(+), 11 deletions(-)

(limited to 'client')

diff --git a/client/src/app/shared/misc/utils.ts b/client/src/app/shared/misc/utils.ts
index 64bc69b0d..d520b1a7b 100644
--- a/client/src/app/shared/misc/utils.ts
+++ b/client/src/app/shared/misc/utils.ts
@@ -59,14 +59,6 @@ function immutableAssign <A, B> (target: A, source: B) {
   return Object.assign({}, target, source)
 }
 
-function isInSmallView () {
-  return window.innerWidth < 600
-}
-
-function isInMobileView () {
-  return window.innerWidth < 500
-}
-
 // Thanks: https://gist.github.com/ghinda/8442a57f22099bdb2e34
 function objectToFormData (obj: any, form?: FormData, namespace?: string) {
   let fd = form || new FormData()
@@ -94,6 +86,18 @@ function lineFeedToHtml (obj: object, keyToNormalize: string) {
   })
 }
 
+// Try to cache a little bit window.innerWidth
+let windowInnerWidth = window.innerWidth
+setInterval(() => windowInnerWidth = window.innerWidth, 500)
+
+function isInSmallView () {
+  return windowInnerWidth < 600
+}
+
+function isInMobileView () {
+  return windowInnerWidth < 500
+}
+
 export {
   viewportHeight,
   getParameterByName,
diff --git a/client/src/app/shared/video/infinite-scroller.directive.ts b/client/src/app/shared/video/infinite-scroller.directive.ts
index 1da02f60f..16e901c34 100644
--- a/client/src/app/shared/video/infinite-scroller.directive.ts
+++ b/client/src/app/shared/video/infinite-scroller.directive.ts
@@ -5,6 +5,7 @@ import 'rxjs/add/operator/distinctUntilChanged'
 import 'rxjs/add/operator/filter'
 import 'rxjs/add/operator/map'
 import 'rxjs/add/operator/startWith'
+import 'rxjs/add/operator/throttleTime'
 import { fromEvent } from 'rxjs/observable/fromEvent'
 
 @Directive({
@@ -37,6 +38,7 @@ export class InfiniteScrollerDirective implements OnInit {
   initialize () {
     const scrollObservable = fromEvent(window, 'scroll')
       .startWith(true)
+      .throttleTime(200)
       .map(() => ({ current: window.scrollY, maximumScroll: document.body.clientHeight - window.innerHeight }))
 
     // Scroll Down
@@ -49,7 +51,6 @@ export class InfiniteScrollerDirective implements OnInit {
         return res
       })
       .filter(({ current, maximumScroll }) => maximumScroll <= 0 || (current / maximumScroll) > this.decimalLimit)
-      .debounceTime(200)
       .distinct()
       .subscribe(() => this.nearOfBottom.emit())
 
@@ -65,13 +66,11 @@ export class InfiniteScrollerDirective implements OnInit {
       .filter(({ current, maximumScroll }) => {
         return current !== 0 && (1 - (current / maximumScroll)) > this.decimalLimit
       })
-      .debounceTime(200)
       .distinct()
       .subscribe(() => this.nearOfTop.emit())
 
     // Page change
     scrollObservable
-      .debounceTime(500)
       .distinct()
       .map(({ current }) => Math.max(1, Math.round((current + InfiniteScrollerDirective.PAGE_VIEW_TOP_MARGIN) / this.pageHeight)))
       .distinctUntilChanged()
-- 
cgit v1.2.3