Skip to content

Fix a crash when removing tokens#25

Open
yeahphil wants to merge 1 commit intojasarien:masterfrom
yeahphil:fix-remove
Open

Fix a crash when removing tokens#25
yeahphil wants to merge 1 commit intojasarien:masterfrom
yeahphil:fix-remove

Conversation

@yeahphil
Copy link

  • textField:shouldChangeCharactersInRange:replacementString: must check if
    the delegate implements tokenField:shouldRemoveToken:representedObject:
    before calling it
  • if the method isn't implemented, the default should be to assume
    tokens can be deleted.

- textField:shouldChangeCharactersInRange:replacementString: must check if
the delegate implements tokenField:shouldRemoveToken:representedObject:
before calling it

- if the method isn't implemented, the default should be to assume
  tokens can be deleted.
@yeahphil
Copy link
Author

Note that without this commit, the example app throws an uncaught exception if you try to remove a token by pressing the backspace key to highlight it.

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