Small refactoring of the loadingBar hiding code

11 years ago · cfeb4c1019
3 changed files with 3 additions and 5 deletions
--- a/web/ui_utils.js
+++ b/web/ui_utils.js
@ -329,7 +329,7 @@ var ProgressBar = (function ProgressBarClosure() {
				@@ -329,7 +329,7 @@ var ProgressBar = (function ProgressBarClosure() {

    hide: function ProgressBar_hide() {
      this.bar.classList.add('hidden');
-      this.bar.removeAttribute('style');
+      document.body.classList.remove('loadingInProgress');
    }
  };

--- a/web/viewer.html
+++ b/web/viewer.html
@ -96,8 +96,8 @@ http://sourceforge.net/adobe/cmap/wiki/License/
				@@ -96,8 +96,8 @@ http://sourceforge.net/adobe/cmap/wiki/License/

  </head>

-  <body tabindex="1">
-    <div id="outerContainer" class="loadingInProgress">
+  <body tabindex="1" class="loadingInProgress">
+    <div id="outerContainer">

      <div id="sidebarContainer">
        <div id="toolbarSidebar">
--- a/web/viewer.js
+++ b/web/viewer.js
@ -873,8 +873,6 @@ var PDFViewerApplication = {
				@@ -873,8 +873,6 @@ var PDFViewerApplication = {
    var downloadedPromise = pdfDocument.getDownloadInfo().then(function() {
      self.downloadComplete = true;
      self.loadingBar.hide();
-      var outerContainer = document.getElementById('outerContainer');
-      outerContainer.classList.remove('loadingInProgress');
    });

    var pagesCount = pdfDocument.numPages;