Merge pull request #341 from xwp/issue/35

Focus on specific field that have conflict

Author: westonruter

js/customize-post-section.js

@@ -891,7 +891,8 @@
 
 			// Detect conflict errors.
 			api.bind( 'error', function( response ) {
-				var theirValue, ourValue;
+				var theirValue, ourValue,
+					conflictedControls = [];
 				if ( ! response.update_conflicted_setting_values ) {
 					return;
 				}
@@ -912,8 +913,22 @@
 						} );
 						control.notifications.remove( notification.code );
 						control.notifications.add( notification.code, notification );
+						conflictedControls.push( control );
 					}
 				} );
+
+				// Focus on first field that have conflict.
+				if ( section.expanded() ) {
+					_.every( section.controls(), function( _setting ) {
+						return _.every( conflictedControls, function( conflictedControl ) {
+							if ( conflictedControl.id === _setting.id ) {
+								_setting.focus();
+								return false;
+							}
+							return true;
+						} );
+					} );
+				}
 			} );
 
 			api.bind( 'save', function() {