Show sticky error message on viewport message listing error

horde · Sep 18, 2014 · bb18b22 · bb18b22
1 parent 5b8621c
commit bb18b22
Show file tree

Hide file tree

Showing 4 changed files with 29 additions and 0 deletions.
diff --git a/imp/js/dimpbase.js b/imp/js/dimpbase.js
@@ -2308,6 +2308,20 @@ var DimpBase = {
         }
     },
 
+    /* Handle viewport errors. */
+    showViewportError: function(show)
+    {
+        if ($('viewport_error').visible()) {
+            if (!show) {
+                $('viewport_error').hide();
+                this.viewport.onResize(true);
+            }
+        } else if (show) {
+            $('viewport_error').show();
+            this.viewport.onResize(true);
+        }
+    },
+
     /* Enable/Disable action buttons as needed. */
     toggleButtons: function()
     {
@@ -3783,6 +3797,7 @@ var DimpBase = {
         }
 
         if (this.viewport && t['imp:viewport']) {
+            this.showViewportError(t['imp:viewport'].error);
             t['imp:viewport'].requestid = e.response.request.rid;
             this.viewport.parseJSONResponse(t['imp:viewport']);
         }

diff --git a/imp/lib/Ajax/Application/Viewport/Error.php b/imp/lib/Ajax/Application/Viewport/Error.php
@@ -48,6 +48,7 @@ public function toObject()
     {
         $ob = new stdClass;
         $ob->cacheid = strval(new Horde_Support_Randomid());
+        $ob->error = true;
         $ob->label = $this->_mbox->label;
         $ob->view = $this->_mbox->form_to;
 

diff --git a/imp/templates/dynamic/mailbox.html.php b/imp/templates/dynamic/mailbox.html.php
@@ -63,6 +63,10 @@
      <span id="search_time_elapsed" style="display:none"></span>
      <div id="search_label"></div>
     </div>
+
+    <div id="viewport_error" style="display:none">
+     <div><?php echo _("Error loading message list.") ?></div>
+    </div>
    </div>
 
    <div id="msgSplitPane"></div>

diff --git a/imp/themes/default/dynamic/screen.css b/imp/themes/default/dynamic/screen.css
@@ -818,6 +818,15 @@ span.attachmentImg {
     padding-right: 12px;
 }
 
+/* Viewport error */
+#viewport_error {
+    background-color: red;
+    padding: 8px 10px;
+}
+#viewport_error div {
+    font-weight: bold;
+}
+
 /* Redbox styles. */
 #RB_window {
     max-width: 50%;