Resolve issue HUDSON-9025.

misunwork · Jan 19, 2012 · 580e263 · 580e263
1 parent 16ec2d8
commit 580e263
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 22 deletions.
diff --git a/hudson-core/src/main/resources/hudson/security/SecurityRealm/loginDialog.jelly b/hudson-core/src/main/resources/hudson/security/SecurityRealm/loginDialog.jelly
@@ -61,22 +61,6 @@ THE SOFTWARE.
             <input style="margin-left:10px; width:75px" type="button" id="cancelButton" value="${%Cancel}" />
 
         </form>
-        <style type="text/css">
-            #loginError, #loginMsg {
-                opacity:0;
-                -ms-filter:"progid:DXImageTransform.Microsoft.Alpha(Opacity=0)";
-                filter: alpha(opacity=0);
-                text-align:center;
-                font-weight:bold;
-            }
-            #loginError {
-                color: red;
-            }
-            #loginMsg {
-                color: black;
-            }
-        </style>
-
         <div id="loginError">
             ${%Invalid login information. Please try again.}
         </div>

diff --git a/hudson-core/src/main/resources/hudson/security/SecurityRealm/loginLink.jelly b/hudson-core/src/main/resources/hudson/security/SecurityRealm/loginLink.jelly
@@ -45,11 +45,13 @@ THE SOFTWARE.
               title:  'Confirmation'
             });
             jQuery('j_username').focus();
+            jQuery('#loginMsg').hide();
+            jQuery('#loginError').hide();
         }
 
         function submitForm(){
-            jQuery('#loginMsg').css({ opacity: 1.0 });
-            jQuery('#loginError').css({ opacity: 0.0 });
+            jQuery('#loginMsg').show();
+            jQuery('#loginError').hide();
             var dataString = jQuery("#loginForm").serialize();
             jQuery.ajax({
               type: 'POST',
@@ -59,8 +61,8 @@ THE SOFTWARE.
                 window.location.href="${from}";
               },
               error: function(){
-                jQuery('#loginError').css({ opacity: 1.0 });
-                jQuery('#loginMsg').css({ opacity: 0.0 });
+                jQuery('#loginError').show();
+                jQuery('#loginMsg').hide();
               },
               dataType: "html"
             }); 
@@ -90,8 +92,8 @@ THE SOFTWARE.
                 jQuery.unblockUI();
                 jQuery('#j_username').attr({value:""});
                 jQuery('#j_password').attr({value:""});
-                jQuery('#loginError').css({ opacity: 0.0 });
-                jQuery('#loginMsg').css({ opacity: 0.0 });
+                jQuery('#loginError').hide();
+                jQuery('#loginMsg').hide();
                 return false;
             });
 

diff --git a/hudson-war/src/main/webapp/css/style.css b/hudson-war/src/main/webapp/css/style.css
@@ -1073,3 +1073,16 @@ table.configure > td {
     border-collapse: collapse;
     width: 450px;
 }
+/* === Login Dialog ==== */
+#loginError, #loginMsg {
+    text-align:center;
+    font-weight:bold;
+}
+#loginError {
+    color: red;
+    display: none;
+}
+#loginMsg {
+    color: black;
+    display: none;
+}