Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support OGG using SoundManager2 #416

Merged
merged 6 commits into from
Sep 3, 2015
Merged

Conversation

pellaeon
Copy link
Member

@pellaeon pellaeon commented Sep 1, 2015

A new wrapper class PlayerWrapper is created, which is used to call underlying Aurora.js or SoundManager2 depending on format.

return AV.Player;
//return AV.Player;
return new PlayerWrapper();
//return PlayerWrapper;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you drop these comments here?

@MorrisJobke
Copy link
Contributor

Beside my comments this looks really awesome :) Thanks for the contribution :)

@scrutinizer-notifier
Copy link

A new inspection was created.

@pellaeon
Copy link
Member Author

pellaeon commented Sep 3, 2015

@MorrisJobke Do you want to merge this PR first or you want to wait after #419 ?

MorrisJobke added a commit that referenced this pull request Sep 3, 2015
Support OGG using SoundManager2
@MorrisJobke MorrisJobke merged commit c9d5306 into owncloud:master Sep 3, 2015
@MorrisJobke MorrisJobke added this to the 0.3.8 milestone Sep 3, 2015
@MorrisJobke MorrisJobke mentioned this pull request Sep 3, 2015
@pellaeon
Copy link
Member Author

pellaeon commented Sep 3, 2015

yay 🎆 🎆

MorrisJobke added a commit that referenced this pull request Oct 27, 2015
 * support for ogg (#416 by pellaeon)
 * fix issue with not existing prepareQuery (#411 by roha4000)
 * fix failures after upload to public link shares (#436, #387)
 * fix for Angular variable names (#425 by DavidPrevot)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants