From 2d28c00d0f457e4bea3059a0f42a6d4168602289 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tim=20Daubensch=C3=BCtz?= <tim@ascribe.io>
Date: Mon, 13 Jul 2015 19:12:24 +0200
Subject: [PATCH] display spinner on pagination request

---
 js/actions/piece_list_actions.js | 15 +++++++++++++++
 js/stores/piece_list_store.js    |  1 +
 2 files changed, 16 insertions(+)

diff --git a/js/actions/piece_list_actions.js b/js/actions/piece_list_actions.js
index ec7ad7a7..3c5b44d2 100644
--- a/js/actions/piece_list_actions.js
+++ b/js/actions/piece_list_actions.js
@@ -14,6 +14,21 @@ class PieceListActions {
     }
 
     fetchPieceList(page, pageSize, search, orderBy, orderAsc) {
+        // To prevent flickering on a pagination request,
+        // we overwrite the piecelist with an empty list before
+        // pieceListCount === -1 defines the loading state
+        this.actions.updatePieceList({
+            page,
+            pageSize,
+            search,
+            orderBy,
+            orderAsc,
+            'pieceList': [],
+            'pieceListCount': -1
+        });
+
+        // afterwards, we can load the list
+
         return new Promise((resolve, reject) => {
             PieceListFetcher
                 .fetch(page, pageSize, search, orderBy, orderAsc)
diff --git a/js/stores/piece_list_store.js b/js/stores/piece_list_store.js
index 63a109ca..22d8d5d6 100644
--- a/js/stores/piece_list_store.js
+++ b/js/stores/piece_list_store.js
@@ -19,6 +19,7 @@ class PieceListStore {
          * the number of items the resource actually - without pagination - provides.
          */
         this.pieceList = [];
+        // -1 specifies that the store is currently loading
         this.pieceListCount = -1;
         this.page = 1;
         this.pageSize = 10;