search

sourcebitsllc / chocolatechip-ui

Mobile Web App Framework
www.chocolatechip-ui.com
MIT License
616 stars 88 forks source link

Actual bugfix for $.UIDeletable #22

Closed grrowl closed 10 years ago

grrowl commented 10 years ago

As long as other parts of ChUI aren't binding to the "edit" and "done" buttons, this is a proper fix.

Sloppy contributing on my part before, this is a snippet from our tested codebase.

sourcebits-robertbiggs commented 10 years ago

Hey Tom,

I just did a total refactor of $.UIDeletable. Now it appears to be working properly with multiple lists in different articles. In the initialization of a deletable list I'm passing in a reference to the button and the list so that lists are directly link to their article-specific edit button. This also solves the issue where a callback was getting used by all the list. Please check out commit b39e3fd3cb and let me know if this covers all your issues. Thanks for the help in unravelling this.

grrowl commented 10 years ago

You legend, will take a look later in the week. Cheers