Ticket #15204: 15204_v2.diff

wp-admin/js/common.dev.js

@@ -180 +180 @@
 $(document).ready(function(){columns.init();});
 
 validateForm = function( form ) {
-        return !$( form ).find('.form-required').filter( function() { return $('input:visible', this).val() == ''; } ).addClass( 'form-invalid' ).find('input:visible').change( function() { $(this).closest('.form-invalid').removeClass( 'form-invalid' ); } ).size();
+        return !$( form )
+                .find('.form-required')
+                        .filter( function() {
+                                return $('input:visible', this).val().trim() === '';
+                        } )
+                        .addClass( 'form-invalid' )
+                        .find('input:visible')
+                                .filter(':first')
+                                        .focus()
+                                        .end()
+                                .change( function() {
+                                        if(this.value.trim() !== '')
+                                                $(this).closest('.form-invalid').removeClass( 'form-invalid' );
+                                } )
+                                .size();
 }
 
 })(jQuery);

wp-includes/js/wp-ajax-response.dev.js

@@ -50 +50 @@
                 return true;
         },
         invalidateForm: function ( selector ) {
-                return jQuery( selector ).addClass( 'form-invalid' ).find('input:visible').change( function() { jQuery(this).closest('.form-invalid').removeClass( 'form-invalid' ); } );
+                return jQuery( selector )
+                        .addClass( 'form-invalid' )
+                        .find('input:visible')
+                                .filter(':first')
+                                        .focus()
+                                        .end()
+                                .change( function() {
+                                        if(this.value.trim() !== '')
+                                                jQuery(this).closest('.form-invalid').removeClass( 'form-invalid' );
+                                } );
         },
         validateForm: function( selector ) {
                 selector = jQuery( selector );
-                return !wpAjax.invalidateForm( selector.find('.form-required').filter( function() { return jQuery('input:visible', this).val() == ''; } ) ).size();
+                return !wpAjax.invalidateForm( 
+                        selector
+                                .find('.form-required')
+                                        .filter( function() {
+                                                return jQuery('input:visible', this).val().trim() === ''; 
+                                        } )
+                ).size();
         }
 }, wpAjax || { noPerm: 'You do not have permission to do that.', broken: 'An unidentified error has occurred.' } );