From 2b851d47f2aa09e4b8535d3b39ed93440f6eee27 Mon Sep 17 00:00:00 2001
From: Marc Englund <marc.englund@itmill.com>
Date: Fri, 6 Jun 2008 08:40:32 +0000
Subject: [PATCH] Fixed multiple bugs in ProgressIndicator: polled even if
 disabled or invisible (#1581), had no isIndeterminate(), did not call
 requestRepaint() when changing polling interval or indeterminate mode.

svn changeset:4771/svn branch:trunk
---
 .../progressindicator/img/disabled.gif        | Bin 0 -> 179 bytes
 .../progressindicator/progressindicator.css   |  13 ++++++++++++
 .../gwt/client/ui/IProgressIndicator.java     |  20 ++++++++++++++++--
 .../itmill/toolkit/ui/ProgressIndicator.java  |  13 +++++++++++-
 4 files changed, 43 insertions(+), 3 deletions(-)
 create mode 100644 WebContent/ITMILL/themes/default/progressindicator/img/disabled.gif

diff --git a/WebContent/ITMILL/themes/default/progressindicator/img/disabled.gif b/WebContent/ITMILL/themes/default/progressindicator/img/disabled.gif
new file mode 100644
index 0000000000000000000000000000000000000000..7a64d034b2e075903853ae437452598c3df18e5d
GIT binary patch
literal 179
zcmZ?wbhEHb<YM4tXpv+%eeS~jM^6qOId=EKqk9h@U%Ybd{G}`FH*LOp<JP1p)2>{<
zdGy4oWh+))zINmAvExgYt=O`C$Fv!<4jnx<edg@rr_LNaa&+nPmB&t=-n?!5e{4YU
zCkrDhgFJ%{0}z1hWMDN&n9_4H#qdmUqI<Q&q{j+pmF^r`s^DlK(V(!vLE%f?A;pgz
UK6<sj4~01o1eTdZb1+x~0I9ZR;{X5v

literal 0
HcmV?d00001

diff --git a/WebContent/ITMILL/themes/default/progressindicator/progressindicator.css b/WebContent/ITMILL/themes/default/progressindicator/progressindicator.css
index 57bc14aba8..669942cc49 100644
--- a/WebContent/ITMILL/themes/default/progressindicator/progressindicator.css
+++ b/WebContent/ITMILL/themes/default/progressindicator/progressindicator.css
@@ -5,6 +5,13 @@
 	overflow: hidden; /* for IE6 */
 }
 
+.i-progressindicator-disabled {
+	background: #dfe2e4 url(img/disabled.gif);
+	height: 9px;
+	border: 1px solid #b6bbbc;
+	overflow: hidden; /* for IE6 */
+}
+
 .i-progressindicator div {
 	background: #f7f9f9 url(img/progress.png);
 	height: 9px;
@@ -16,4 +23,10 @@
 	height: 16px;
 	width: 16px;
 	overflow: hidden; /* for IE6 */
+}
+.i-progressindicator-disabled-indeterminate {
+	background: #dfe2e4 url(../common/img/blank.gif);
+	height: 16px;
+	width: 16px;
+	overflow: hidden; /* for IE6 */
 }
\ No newline at end of file
diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ui/IProgressIndicator.java b/src/com/itmill/toolkit/terminal/gwt/client/ui/IProgressIndicator.java
index 5845599cf5..c7c062c138 100644
--- a/src/com/itmill/toolkit/terminal/gwt/client/ui/IProgressIndicator.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ui/IProgressIndicator.java
@@ -41,9 +41,15 @@ public class IProgressIndicator extends Widget implements Paintable {
 
         indeterminate = uidl.getBooleanAttribute("indeterminate");
 
+        String style = CLASSNAME;
+        if (uidl.getBooleanAttribute("disabled")) {
+            style += "-disabled";
+        }
+
         if (indeterminate) {
-            this.setStyleName(CLASSNAME + "-indeterminate");
+            this.setStyleName(style + "-indeterminate");
         } else {
+            setStyleName(style);
             try {
                 final float f = Float.parseFloat(uidl
                         .getStringAttribute("state"));
@@ -52,7 +58,17 @@ public class IProgressIndicator extends Widget implements Paintable {
             } catch (final Exception e) {
             }
         }
-        poller.scheduleRepeating(uidl.getIntAttribute("pollinginterval"));
+
+        if (!uidl.getBooleanAttribute("disabled")) {
+            poller.scheduleRepeating(uidl.getIntAttribute("pollinginterval"));
+        }
+    }
+
+    public void setVisible(boolean visible) {
+        super.setVisible(visible);
+        if (!visible) {
+            poller.cancel();
+        }
     }
 
     class Poller extends Timer {
diff --git a/src/com/itmill/toolkit/ui/ProgressIndicator.java b/src/com/itmill/toolkit/ui/ProgressIndicator.java
index 8a48239da0..c10ac7daca 100644
--- a/src/com/itmill/toolkit/ui/ProgressIndicator.java
+++ b/src/com/itmill/toolkit/ui/ProgressIndicator.java
@@ -214,13 +214,23 @@ public class ProgressIndicator extends AbstractField implements Property,
     }
 
     /**
-     * Sets the ProgressIndicator to indeterminate mode.
+     * Sets wheter or not the ProgressIndicator is indeterminate.
      * 
      * @param newValue
      *                true to set to indeterminate mode.
      */
     public void setIndeterminate(boolean newValue) {
         indeterminate = newValue;
+        requestRepaint();
+    }
+
+    /**
+     * Gets whether or not the ProgressIndicator is indeterminate.
+     * 
+     * @return true to set to indeterminate mode.
+     */
+    public boolean isIndeterminate() {
+        return indeterminate;
     }
 
     /**
@@ -231,6 +241,7 @@ public class ProgressIndicator extends AbstractField implements Property,
      */
     public void setPollingInterval(int newValue) {
         pollingInterval = newValue;
+        requestRepaint();
     }
 
     /**
-- 
2.39.5