Ticket #15204: 15204.diff

wp-admin/js/common.dev.js

@@ -169 +169 @@
 
 $(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();
-}
-
 })(jQuery);
 
 // stub for doing better warnings

wp-admin/js/tags.dev.js

@@ -29 +29 @@
         $('#submit').click(function(){
                 var form = $(this).parents('form');
 
-                if ( !validateForm( form ) )
+                if ( !wpAjax.validateForm( form ) )
                         return false;
 
                 $.post(ajaxurl, $('#addtag').serialize(), function(r){

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(jQuery.trim(this.value)) 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.' } );