Guillaume Potier

Results 73 comments of Guillaume Potier

As promised, here a first glance and comments at your PR. Also, to be merged, after updating your PR accordingly to my comments, you'll need to: - [ ] fix...

Hi. Thanks for opening this issue! Could you try to put the listener call `this.options.onRetrieve( this.$element, storedValue );` (L170) at line 152, before checkbox and radio test ? `if (...

Could you please make a jsbin or jsfiddle showing this behavior ? Thanks

You're right. Could you please replace `attr` by adequate `prop` and submit a Pull Request ? Thanks

Maybe using iCheck [Callbacks](https://github.com/damirfoy/iCheck#callbacks) ?

As you see [here](https://github.com/guillaumepotier/Garlic.js/blob/master/garlic.js#L94), `submit` and `reset` events are binded to `destroy` function. So, that's strange that a `$('#field').garlic('destroy');` on each field did not solve your case..

If you do not put a `data-persist` on the `` elem, but instead, on every single field you want to persist, that should work.

Hi!, thanks for this issue! Could you run the test-suite too on your Firefox (garlicjs.org/tests/index.html) and paste here the result? Thanks

Strange, I've downloaded FF17 and tested, checkboxes are working fine with me, but some tests are failing in the test suite #20

There is currently a known bug on IE8 https://github.com/guillaumepotier/Garlic.js/issues/5 where it should work. For now, I did not added support for IE8-, maybe I'll implement it later. Best