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

Request for New Target: Chromebook #1329

Closed
gary9872 opened this issue Mar 3, 2018 · 10 comments
Closed

Request for New Target: Chromebook #1329

gary9872 opened this issue Mar 3, 2018 · 10 comments
Labels
💡 enhancement This issue describes an improvement, enhancement, or feature request for OpenShot

Comments

@gary9872
Copy link

gary9872 commented Mar 3, 2018

I was helping out someone recently with some video work and they presentations were at school. Neither of the two projects worked (the audio). The culprit was the chromebooks, which was the standard I believe throughout that entire school system.

So to that end I am requesting to add Chromebook under Device > Target >

using webm VP8 or VP9 for video, and Vorbis for audio as per google support page.

https://support.google.com/chromebook/answer/183093?hl=en

If you like I can submit a diff patch against 2.4.1 or git for it

Thanks,

Gary

@peanutbutterandcrackers
Copy link
Contributor

@DylanC - I think you should take a look at this. He has made an offer we can not refuse.

@gary9872 - Hey! Thanks! I'm so glad that you are helping OpenShot become a better Video Editor. Please do stick around and help us out with other bugs and stuffs too. We need more developers here. Currently, it's just Mr. Thomas alone. We really appreciate the help. :)

@DylanC
Copy link
Collaborator

DylanC commented Mar 3, 2018

@gary9872 - If you could submit a PR that would be great! Might take a while to get it reviewed but at least it will get in after a while.

@peanutbutterandcrackers
Copy link
Contributor

Yeah! You go, @gary9872 !!! 😄

@DylanC DylanC added the 💡 enhancement This issue describes an improvement, enhancement, or feature request for OpenShot label Mar 4, 2018
@gary9872
Copy link
Author

gary9872 commented Mar 5, 2018

Add to openshot-qt/src/presets directory

chromebook.xml

https://pastebin.com/G6sHeCRh

@peanutbutterandcrackers
Copy link
Contributor

@gary9872 - Actually, if you could make a PR, that would probably land you a credit (at least in the git logs) of doing so... Are you sure you do not want to make a PR?

Also, could you please, if you decide to make a PR, take care of #201 ?

Thanks!

@DylanC
Copy link
Collaborator

DylanC commented Mar 6, 2018

I'm adding the file here in case the pastebin disappears or anything like that. (Note its supposed to be just .xml but for uploading purposes I added the .txt to the end.
chromebook.xml.txt

@peanutbutterandcrackers - I can do a PR for this if you like.

@peanutbutterandcrackers
Copy link
Contributor

@DylanC - Well, if @gary9872 does not want to, then please, by all means. :)
Would you like to take care of #201, too, please?

@DylanC
Copy link
Collaborator

DylanC commented Mar 6, 2018

@peanutbutterandcrackers , @gary9872 - PR added. I should be able to get around to #201 too.

@peanutbutterandcrackers
Copy link
Contributor

@DylanC - Awesome!

@gary9872 - I hope you stick around and make some PRs too, good sir!
25vxxj

@DylanC
Copy link
Collaborator

DylanC commented Mar 27, 2018

Closing since this fix has been merged in.

@DylanC DylanC closed this as completed Mar 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 enhancement This issue describes an improvement, enhancement, or feature request for OpenShot
Projects
None yet
Development

No branches or pull requests

3 participants