@@ -40,10 +40,11 @@
}
}
});
- $('#examples li:nth(6)').editable({ save: function() {
- alert('About to save');
- return true;
- } });
+ $('#examples li:nth(6)').editable({
+ validation: function(val) {
+ return confirm('Allow saving?');
+ }
+ });
});
</script>
</head>
@@ -65,7 +66,7 @@
<li>Triggers on Double Click, use buttons to save + cancel</li>
<li><span>Triggers on Double Click, syncs with input</span> <input style="font-size: 0.4em" value="" type="text" disabled /></li>
<li><span>Triggers on Double Click, uses buttons</span> <input style="font-size: 0.4em" value="" type="text" disabled /></li>
- <li>Custom save function (Double Click)</li>
+ <li>Custom Validation</li>
</ol>
</div>
</div>
index.html
@@ -26,23 +26,24 @@ $.widget('ui.editable', {
$input.keydown(function(e) {
if( elem.editable('option', 'finishOnKey') && e.keyCode == elem.editable('option', 'finishOnKey')) $(this).trigger('blur', true);
});
+ elem.trigger(this.widgetEventPrefix + 'Start');
}
},
finish: function() {
var elem = this.element;
- if( elem.data('editing') && ( !elem.editable('option', 'validation') || (elem.editable('option', 'validation') && elem.editable('option', 'validation').test( elem.find('input').val() ))) ) {
- if(this.options.sync) {
- $(this.options.sync).val( elem.find('input').val() );
- elem.text( elem.find('input').val() );
- elem.data('editing', false);
- } else {
- /*
- TODO Fix double save when using blur & enter
- */
- if( this.options.save() ) {
+ if( elem.data('editing') ) {
+ var validation = elem.editable('option', 'validation');
+ var val = elem.find('input').val();
+ if( ($.isFunction( validation ) && validation(val)) || ($.isFunction(validation.test) && validation.test(val)) || !validation) {
+ if(this.options.sync) {
+ $(this.options.sync).val( elem.find('input').val() );
+ elem.text( elem.find('input').val() );
+ elem.data('editing', false);
+ } else {
elem.text( elem.find('input').val() );
elem.data('editing', false);
}
+ elem.trigger(this.widgetEventPrefix + 'Finish');
}
}
},
@@ -51,6 +52,7 @@ $.widget('ui.editable', {
if( elem.data('editing') ) {
elem.text( elem.find('span').attr('title') );
elem.data('editing', false);
+ elem.trigger(this.widgetEventPrefix + 'Cancel');
}
},
_buildButtons: function() {
@@ -67,6 +69,7 @@ $.widget('ui.editable', {
$.extend($.ui.editable, {
version: "0.1",
+ eventPrefix: 'edit',
defaults: {
finishOnKey: 13,
finishOnBlur: true,
@@ -75,8 +78,7 @@ $.extend($.ui.editable, {
eventStart: 'dblclick',
validation: false,
buttons: {},
- sync: false,
- save: function() { return true; }
+ sync: false
}
});
js/ui.editable.js
8ac32e9c17178b2ff8694aa8864c29daa61bdaa1
Stuart Loxton
stuart.loxton@googlemail.com
http://github.com/stuartloxton/jquery-ui-editable/commit/634913288d71d8ff192c352843ea16abe46d87f3
634913288d71d8ff192c352843ea16abe46d87f3
2009-04-14T06:16:23-07:00
2009-04-14T05:47:44-07:00
Removed save config and instead allows validation to be a function with the val as the first param
118a2cdc7f31639481d85dcd3eb5581ab843b7fa
Stuart Loxton
stuart.loxton@googlemail.com