Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

We should add @slides variable #14

Open
ezhlobo opened this issue Jan 8, 2013 · 0 comments
Open

We should add @slides variable #14

ezhlobo opened this issue Jan 8, 2013 · 0 comments
Assignees
Milestone

Comments

@ezhlobo
Copy link
Contributor

ezhlobo commented Jan 8, 2013

Now we use @slides method to get collection of slides. We must add @slides variable instead method to cache it collection. Work should looks like:

slider = new Flow 'slider'

slider.slides # return collection of all slides

slider.append '<li>new slide</li>'
slider.slides # return collection of all slides after appending

slider.list.append '<li>new slide</li>'
slider.slides # and after native appending this should return coollection of all slides

It's so necessary because jquery.find() does not cache result and every call @slides method will be very slow.

@ghost ghost assigned ezhlobo Jan 15, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant