Added comments on workarounds

kiwix · Oct 15, 2018 · c222919 · c222919
1 parent a363a93
commit c222919
Showing 1 changed file with 3 additions and 0 deletions.
diff --git 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);