Skip to content
This repository was archived by the owner on Oct 22, 2020. It is now read-only.

Added the ability to get notified when the Picker gets dismissed via a protocol#5

Open
eddieespinal wants to merge 4 commits intoblackmirror-media:developfrom
eddieespinal:develop
Open

Added the ability to get notified when the Picker gets dismissed via a protocol#5
eddieespinal wants to merge 4 commits intoblackmirror-media:developfrom
eddieespinal:develop

Conversation

@eddieespinal
Copy link
Copy Markdown

I think this featured was needed as I had to performed some cleanup on my viewcontroller if the user dismissed the Picker. If the Save button was pressed it was easy to clean up but not when a user press the Cancel button.

Hopefully someone else will find this useful.

…delegate or when calling dismiss() manually via closure
…sed via delegate or when calling dismiss() manually via closure"

This reverts commit e6d75aa.
…delegate or when calling dismiss() manually via closure
… since we have a delegate that does the same thing
@eriadam
Copy link
Copy Markdown
Member

eriadam commented Jul 13, 2016

Thanks Eddie. It's a great idea.
However, I am trying to avoid the protocol pattern in favour of the closures. The component is already using a closure for handling the selection.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants