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

numberOfVisibleItems #27

Closed
ronnyf opened this issue Aug 12, 2011 · 2 comments
Closed

numberOfVisibleItems #27

ronnyf opened this issue Aug 12, 2011 · 2 comments

Comments

@ronnyf
Copy link

ronnyf commented Aug 12, 2011

error: assigning to property with 'readonly' attribute not allowed [3]

in v. 1.5.2 this was no issue, could you please fix that?

@nicklockwood
Copy link
Owner

It's not a bug - check the documentation. The reason is that numberOfVisibleItems is now set using the carousel: numberOfVisibleItems: dataSource method.

The reason I changed it was to avoid having an arbitrary default limit. Plus, it's more consistent since numberOfItems and numberOfPlaceholders are both set that way.

@ronnyf
Copy link
Author

ronnyf commented Aug 12, 2011

thanks for clarifying... I changed my code and it works now as before.

thanks a lot for this great class!

./Ronny

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants