joshtronic/pickles
1
0
Fork 0
Code Issues 8 Pull requests Projects Releases Packages Wiki Activity Actions

Updated show/hide logic on form submission.

Browse source 
* Instead of hiding the elements I'm marking them as readonly.

 * Refactored the logic to apply read/write access at the end of the block so there's no duplicate code.
This commit is contained in:
Josh Sherman 2010-03-21 11:33:12 -04:00
parent ecff6637fa
commit eb1fd8e364
1 changed files with 7 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

16
js/core.js
View file

@ -11,7 +11,7 @@ $(document).ready(function()
if ($('form').valid() == true)
{
// Hides the buttons on the form
$('button, input', form).hide();
$('button, input', form).attr('readonly', 'readonly');
// @todo Hide the form and add a throbber
@ -47,15 +47,13 @@ $(document).ready(function()
}
else if (data.status == 'success')
{
$('input[type=text]', form).val('');
$('select', form).val('');
$('textarea', form).val('');
if (typeof(data.message) != 'undefined')
{
$('input[type=text]', form).val('');
$('select', form).val('');
$('textarea', form).val('');
alert(data.message);
$('button, input', form).show();
}
if (typeof(data.url) != 'undefined')
@ -66,14 +64,14 @@ $(document).ready(function()
if (typeof(data.callback) != 'undefined')
{
window[data.callback](data);
$('button', form).show();
}
}
else
{
alert('Error: ' + data);
$('button, input', form).show();
}
$('button, input', form).attr('readonly', '');
},
'error': function(XMLHttpRequest, textStatus, errorThrown)