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

Some changes #31

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

Some changes #31

wants to merge 16 commits into from

Conversation

andlister
Copy link

Hi

I have chosen your library for the work I am doing at Foursquare, but I have made some changes to support what we needed out of it. I have tried not to make them foursquare specific in case you are interested in pulling the changes in.

Some things I have added.

  • ARC compatible
  • You can add any UIView as a token
  • Some extra callback methods
  • errr others I cannot remember (check commits)

You probably want to look through the commits before you decide to pull them in as I may have taken it a bit too in being specific for our use case? (I hope not)

Thanks

Andy

@massdonati
Copy link

very nice changes!! hope it'll be merged soon

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.

4 participants