Changeset 280 for trunk/hiptop/pester/net/sabi/pester/AlarmSetDialog.java

Timestamp:

07/24/06 07:19:50 (18 years ago)

Author:

Nicholas Riley

Message:

If you're editing an alarm, focus on the time/interval edit field
rather than the message. Relayout set alarm display when time/date
format changes (yay positionDynamically!). Attempt to fix tooltip for

1-minute alarm refreshing/flashing without changing (needs more

testing). Build against 3.0 libraries by default.

File:

• trunk/hiptop/pester/net/sabi/pester/AlarmSetDialog.java (modified) (4 diffs)

trunk/hiptop/pester/net/sabi/pester/AlarmSetDialog.java

@@ -23 +23 @@
 import danger.ui.StaticTextBox;
 import danger.ui.TypeAheadTextField;
+import danger.ui.View;
 import danger.util.DEBUG;
 
@@ -136 +137 @@
         Application.registerForEvent(this, Event.EVENT_TIME_CHANGED);
         show();
+        if (!asNew) setFocusedDescendant(isIn() ? (View)mPeriodField :
+                                         (View)mTimeEditor);
         if (!mHaveShownWindow) {
             Rect alarmRect = mSetAlarmButton.getFrame();
@@ -145 +148 @@
             mHaveShownWindow = true;
             setAutoSize(false); // don't resize to contain it next time
+            // XXX still a minor potential race here, but it's so unlikely
+            // as to not be worth fixing
+            Application.registerForEvent(this, Event.EVENT_TIME_FORMAT_CHANGED);
         }
     }
@@ -244 +250 @@
             stopEditing();
             return true;
+        case Event.EVENT_TIME_FORMAT_CHANGED: // time format changed
+            mTimeEditor.setFormat(DateTimeEditor.FORMAT_TIME_LONG);
+            return true;
         case Event.EVENT_TIME_CHANGED: // time on device changed
         case Event.EVENT_ALARM: // computed time invalid