From 08effa64bf1eda01d543d7ecf2f821868230b698 Mon Sep 17 00:00:00 2001 From: Axel Uhl Date: Tue, 28 Feb 2012 20:58:04 +0100 Subject: [PATCH] added comments why no event is fired when we set min/max on the slider bar --- .../main/java/com/sap/sailing/gwt/ui/client/RaceTimePanel.java | 2 +- .../src/main/java/com/sap/sailing/gwt/ui/client/TimePanel.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/client/RaceTimePanel.java b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/client/RaceTimePanel.java index aa6aadb0498..830d5230363 100644 --- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/client/RaceTimePanel.java +++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/client/RaceTimePanel.java @@ -174,7 +174,7 @@ public class RaceTimePanel extends TimePanel implements R } // never reduce max if it was already set if (min != null && max != null && (getMax() == null || getMax().before(max))) { - setMinMax(min, max, false); + setMinMax(min, max, /* fireEvent */ false); // no event because we guarantee time to be between min and max } } diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/client/TimePanel.java b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/client/TimePanel.java index 72f6c5f3d27..81dd470abbe 100755 --- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/client/TimePanel.java +++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/client/TimePanel.java @@ -271,7 +271,7 @@ public class TimePanel extends FormPanel implements // Handle it equally for replay and live mode for robustness reasons. This at least allows a user // to watch on even if the time panel was off in its assumptions about race end and end of tracking. if (time.after(getMax())) { - setMinMax(getMin(), time, false); + setMinMax(getMin(), time, /* fireEvent */ false); // no event because we guarantee that time is between min/max } long t = time.getTime(); sliderBar.setCurrentValue(new Double(t), false);