Skip to content

feat(android): add video select support for android platform. #290

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

Closed
wants to merge 5 commits into from

Conversation

mylmz10
Copy link

@mylmz10 mylmz10 commented Jul 29, 2019

PR Checklist

What is the current behavior?

Videos tabs not shown on picker area.

What is the new behavior?

Videos tabs shown on picker area.
Implements #175, #60.

@cla-bot
Copy link

cla-bot bot commented Jul 29, 2019

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign the CLA at https://www.nativescript.org/cla.
CLA has not been signed by users: @mylmz10.
After signing the CLA, you can ask me to recheck this PR by posting @cla-bot check as a comment to the PR.

@tgpetrov
Copy link
Contributor

@mylmz10
Thank you for the PR.
We have a requirement that each contributor signs our CLA. Would it be ok with you to fill out this form?

@mylmz10
Copy link
Author

mylmz10 commented Jul 30, 2019

@tgpetrov I fill out the form.

@cla-bot cla-bot bot added the cla: yes label Jul 30, 2019
@elena-p
Copy link
Contributor

elena-p commented Aug 5, 2019

@cla-bot check

@cla-bot
Copy link

cla-bot bot commented Aug 5, 2019

The cla-bot has been summoned, and re-checked this pull request!

@DimitarTodorov
Copy link
Contributor

Hi @mylmz10,

I've created the same PR, from a branch in our repo, since the CI test will not pass on PR from outside the repo due to security reasons. We will proceed with merge and will notify you once the the feature is released.

@mylmz10
Copy link
Author

mylmz10 commented Aug 24, 2019

@DimitarTodorov I noticed there was one missing and I wanted to be a contributor but it was not. I see that merged and published your commit, thanks(!).

xuhcc added a commit to xuhcc/nativescript-imagepicker that referenced this pull request Aug 27, 2019
…on Android

Currently everything except images and videos is not selectable (text for example).
This commit fixes it and partly reverts changes made in NativeScript#290 and NativeScript#291.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants