Merge pull request trentrichardson#725 from emlun/redirect-focus-option

Add altRedirectFocus option

Author: trentrichardson

.jshintrc

@@ -10,6 +10,5 @@
 	"unused": true,
 	"boss": true,
 	"eqnull": true,
-	"node": true,
-	"es5": true
+	"node": true
 }

src/docs/options.html

@@ -68,6 +68,9 @@ <h3>Alt Field Options</h3>
 
 		<dt>altTimeFormat</dt>
 			<dd><em>Default: (timeFormat option)</em> - The time format to use with the altField.</dd>
+
+		<dt>altRedirectFocus</dt>
+			<dd><em>Default: true</em> - Whether to immediately focus the main field whenever the altField receives focus. Effective at construction time only, changing it later has no effect.</dd>
 	</dl>
 
 	<h3>Timezone Options</h3>

src/jquery-ui-timepicker-addon.js

@@ -99,6 +99,7 @@
 			altTimeFormat: null,
 			altSeparator: null,
 			altTimeSuffix: null,
+			altRedirectFocus: true,
 			pickerTimeFormat: null,
 			pickerTimeSuffix: null,
 			showTimepicker: true,
@@ -271,11 +272,14 @@
 			tp_inst.$input = $input;
 
 			if (tp_inst._defaults.altField) {
-				tp_inst.$altInput = $(tp_inst._defaults.altField).css({
-					cursor: 'pointer'
-				}).focus(function () {
-					$input.trigger("focus");
-				});
+				tp_inst.$altInput = $(tp_inst._defaults.altField);
+				if (tp_inst._defaults.altRedirectFocus === true) {
+					tp_inst.$altInput.css({
+						cursor: 'pointer'
+					}).focus(function () {
+						$input.trigger("focus");
+					});
+				}
 			}
 
 			if (tp_inst._defaults.minDate === 0 || tp_inst._defaults.minDateTime === 0) {

test/.jshintrc

@@ -15,6 +15,7 @@
 		"jQuery",
 		"$",
 		"QUnit",
+		"jasmine",
 		"module",
 		"test",
 		"asyncTest",

test/jquery-ui-timepicker-addon_spec.js

@@ -661,4 +661,39 @@ describe('datetimepicker', function() {
 			});
 		});
 	});
+
+	describe('altField', function() {
+		var $input;
+		var $altField;
+		var inputFocusSpy;
+
+		beforeEach(function() {
+			$input = affix('input');
+			$altField = affix('input');
+
+			inputFocusSpy = jasmine.createSpy();
+			$input.focus(inputFocusSpy);
+		});
+
+		it('should redirect focus to main field', function() {
+			$input.datetimepicker({
+				showOn: 'button',
+				altField: $altField,
+			});
+
+			$altField.trigger('focus');
+			expect(inputFocusSpy).toHaveBeenCalled();
+		});
+
+		it('should not redirect focus to main field if altRedirectFocus is false', function() {
+			$input.datetimepicker({
+				showOn: 'button',
+				altField: $altField,
+				altRedirectFocus: false,
+			});
+
+			$altField.trigger('focus');
+			expect(inputFocusSpy).not.toHaveBeenCalled();
+		});
+	});
 });