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

Enhancement (Video support) #1

Closed
solcre-gr opened this issue Dec 9, 2019 · 10 comments
Closed

Enhancement (Video support) #1

solcre-gr opened this issue Dec 9, 2019 · 10 comments
Labels
enhancement New feature or request

Comments

@solcre-gr
Copy link

There is any plans to add videos in the gallery?

@kolkov
Copy link
Owner

kolkov commented Dec 9, 2019

Hi! Thanks for your request.
Do you want to use video from youtube or from other resources?

@solcre-gr
Copy link
Author

Hi!, thank u for your response, i need to use it with local videos, like .mp4 or .avi.

@kolkov kolkov added the enhancement New feature or request label Dec 9, 2019
@hverma
Copy link

hverma commented Feb 1, 2020

+1

@hverma
Copy link

hverma commented Feb 22, 2020

@kolkov wondering if you have a targate date on this enhancement. Thx in advance!

@kolkov
Copy link
Owner

kolkov commented Feb 22, 2020

Thanks for your interesting. I finishing a big project at next week, after that I want to do this enchancement.
If you have a PR, it's speed up my work.

@Simbaclaws
Copy link
Contributor

Simbaclaws commented Apr 17, 2020

@kolkov

You forked the project from lukasz-galka's project right?
In which case you saw the pull request that Muhammad Altabba made.

If you could kindly add his code to your repository that would be great.

His code can be found here:
https://github.com/lukasz-galka/ngx-gallery/pull/257/files/b5d80e13a9e376d1065497a464b126f72066e2c1#diff-90871a8f9a0306e34e7bee9f707e475e

In his pull request for lukasz-galka's project.

Please make this work, I can't seem to find any angular gallery package that supports video currently.

EDIT: if it's hard to read the diff changes, what you could do is clone the repository from lukasz-galka and then add the pull request yourself with git, so you can actually look through the code like you would without having to look at the diffs inside the above thread. This might be easier in figuring out where to add this code into your angular 8 version of this project.

If you need help with making the pull request, I might be able to find some time to do this later on. Please let me know if you're having issues with this and I'll help out as much as I can.

@kolkov
Copy link
Owner

kolkov commented Apr 17, 2020

Hi! It's not a fork, but 100% compatible by API. I will try to implement that PR into this project.
Thanks.

@Simbaclaws
Copy link
Contributor

I added video support to my pull request here:
#23

@Simbaclaws
Copy link
Contributor

Pull Request has been merged, I suppose this issue can be set to solved. I don't have the ability to do so though.

@kolkov
Copy link
Owner

kolkov commented Apr 17, 2020

@Simbaclaws thanks for the PR. Try latest version please.

@kolkov kolkov closed this as completed Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants