From 6221dc9a536c6632e4969c48cd58474350ea1b12 Mon Sep 17 00:00:00 2001 From: Jack Danger Canty Date: Tue, 25 Aug 2009 15:54:01 -0700 Subject: [PATCH 1/2] removing trailing whitespace --- jquery.jeditable.timepicker.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/jquery.jeditable.timepicker.js b/jquery.jeditable.timepicker.js index 3b7dd27..eafcb45 100644 --- a/jquery.jeditable.timepicker.js +++ b/jquery.jeditable.timepicker.js @@ -5,7 +5,7 @@ * * Licensed under the MIT license: * http://www.opensource.org/licenses/mit-license.php - * + * * Depends on Timepicker jQuery plugin by Jason Huck: * http://jquery.com/plugins/project/timepicker * @@ -15,9 +15,9 @@ * Revision: $Id$ * */ - + $.editable.addInputType('timepicker', { - /* This uses default hidden input field. No need for element() function. */ + /* This uses default hidden input field. No need for element() function. */ /* Call before submit hook. */ submit: function (settings, original) { @@ -27,7 +27,7 @@ $.editable.addInputType('timepicker', { $('input', this).val(value); }, /* Attach Timepicker plugin to the default hidden input element. */ - plugin: function(settings, original) { + plugin: function(settings, original) { $('input', this).filter(':hidden').timepicker(); } -}); +}); \ No newline at end of file From e8f57dcc53d102544e1fea45ef586522c18c3d50 Mon Sep 17 00:00:00 2001 From: Jack Danger Canty Date: Tue, 25 Aug 2009 16:07:36 -0700 Subject: [PATCH 2/2] making sure all jquery timepicker inputs have ids The lack of an id resulted in opaque and debilitating errors. jQuery.jeditable doesn't give it's form inputs an id but the timepicker plugin requires one. --- jquery.jeditable.timepicker.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/jquery.jeditable.timepicker.js b/jquery.jeditable.timepicker.js index eafcb45..ec7d61d 100644 --- a/jquery.jeditable.timepicker.js +++ b/jquery.jeditable.timepicker.js @@ -15,7 +15,7 @@ * Revision: $Id$ * */ - +var timepickerFormId = 0 $.editable.addInputType('timepicker', { /* This uses default hidden input field. No need for element() function. */ @@ -28,6 +28,8 @@ $.editable.addInputType('timepicker', { }, /* Attach Timepicker plugin to the default hidden input element. */ plugin: function(settings, original) { - $('input', this).filter(':hidden').timepicker(); + $('input', this).filter(':hidden') + .attr("id", "jquery_timepicker_"+(++timepickerFormId)) + .filter(':hidden').timepicker(); } }); \ No newline at end of file