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

Adding delete option #12

Closed
wants to merge 1 commit into from
Closed

Conversation

Applitom
Copy link

@Applitom Applitom commented Aug 4, 2015

Hi,
I am using CZPicker, it is really great component for my App.
I was needed to add an option to delete row from the list so i've added it.
It would be great if you will add it to the main branch so everybody will be able to use this option :)

Thanks!

@chenzeyu
Copy link
Owner

chenzeyu commented Aug 6, 2015

Hi @Applitom , thanks for your pull request.

Everything is working well in the demo.

However I wonder in what circumstances would you want to delete an item from a picker?

@Applitom
Copy link
Author

Applitom commented Aug 6, 2015

In my Remote control app the user have the option to save and load playlists.
I'm using CZPicker to show to the user his Saved Playlists, and i want to have the option to delete Saved playlist if the user want to.
gif

@chenzeyu
Copy link
Owner

Can you give a animation after delete the row and then dismiss the picker? I think currently it was dismissed to quickly.

@chenzeyu chenzeyu closed this Mar 24, 2016
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.

None yet

3 participants