Changeset 116 for trunk/common/Dialog.js

Timestamp:

02/01/07 22:38:21 (3 years ago)

Author:

ydnar

Message:

added message support to Dialog.Simple; going to phase out Dialog.Message and Dialog.Simple in favor of just Dialog in a future commit; bugid:47041

Files:

• trunk/common/Dialog.js (modified) (5 diffs)

trunk/common/Dialog.js

@@ -98 +98 @@
     open: function() {
         Dialog.Message.superClass.open.apply( this, arguments );
-        var messageElement = DOM.getElement( this.element.id + "-message" );
-        if ( typeof this.data == 'string' )
+        if( typeof this.data == "string" )
             this.data = { message: this.data };
-        
-        if( messageElement )
-            messageElement.innerHTML = this.data.message;
+        var e = DOM.getElement( this.element.id + "-message" );
+        if( e )
+            e.innerHTML = this.data.message;
     }
 } );
@@ -208 +207 @@
 
     setFirstInput: function() {
-        if ( !this.firstInput )
+        if( !this.firstInput )
             this.firstInput = this.getFirstSpecifiedInput( this.element, this.data, null );
     },
@@ -214 +213 @@
 
     selectFirstInput: function() {
-        if ( this.firstInput && this.firstInput.select )
+        if( this.firstInput && this.firstInput.select )
             this.firstInput.select();
     },
@@ -243 +242 @@
     open: function( data, callback ) {
         Dialog.Simple.superClass.open.apply( this, arguments );
-        if( !this.data ) 
-            this.data = {};
+        if( typeof this.data == "string" )
+            this.data = { message: this.data };
+        var e = DOM.getElement( this.element.id + "-message" );
+        if( e )
+            e.innerHTML = this.data.message;
+        
         this.firstInput = null;
         DOM.setFormData( this.element, this.data );
@@ -250 +253 @@
         this.focusFirstInput();
     }
-
 } );
-
-