From c222919f34e2e452f8e723bcd8f196aa8e208402 Mon Sep 17 00:00:00 2001
From: Jaifroid <egk10@cam.ac.uk>
Date: Mon, 15 Oct 2018 21:23:25 +0100
Subject: [PATCH] Added comments on workarounds

---
 www/js/app.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/www/js/app.js b/www/js/app.js
index ac084ffb0..13e7f2934 100644
--- a/www/js/app.js
+++ b/www/js/app.js
@@ -751,6 +751,8 @@ define(['jquery', 'zimArchiveLoader', 'util', 'uiUtil', 'cookies','abstractFiles
             if (dirEntry.isRedirect()) {
                 selectedArchive.resolveRedirect(dirEntry, readArticle);
             } else {
+                // Line below was inserted to prevent the spinner being hidden, possibly by an async function, when pressing the Random button in quick succession
+                // TODO: Investigate whether it is really an async issue or whether there is a rogue .hide() statement in the chain
                 $("#searchingArticles").show();
                 selectedArchive.readUtf8File(dirEntry, displayArticleContentInIframe);
             }
@@ -997,6 +999,7 @@ define(['jquery', 'zimArchiveLoader', 'util', 'uiUtil', 'cookies','abstractFiles
                     $('#cachingCSS').hide();
                     $('#searchingArticles').hide();
                     $('#articleContent').show();
+                    // We have to resize here for devices with On Screen Keyboards when loading from the article search list
                     resizeIFrame();
                 } else if (title) {
                     title = title.replace(/[^/]+\//g, '').substring(0,18);