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

Additions #26

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Additions #26

wants to merge 6 commits into from

Conversation

nobre84
Copy link

@nobre84 nobre84 commented May 12, 2015

Hi! This is a good component, thanks for sharing!
I needed some tweaks and fixes for my use case, and hope it might be useful for the library or anyone using it:

• Doesn't require FB on the Demo - only request authorization in the FB tab
• Uncoupled Items based on a protocol instead of requiring a strict item class (allow for showing arbitrary model objects directly, either by implementing the protocol in your class or in a category)
• Delegate block helper if one wants to move away from delegation pattern
• Some typos & warnings fixed

… on it instead of the concrete interface.

Added a quick bit mask selection option in addition to the selected items array.
The cancelDelegate should include the current Selector (maybe should deprecate the old one and promote the new one as required).
Created a helper class to allow using blocks as the delegate to the selector.
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

Successfully merging this pull request may close these issues.

1 participant