-
Notifications
You must be signed in to change notification settings - Fork 23
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
VP-211: subtitlesFinder component for a VideoPlayer #125
base: main
Are you sure you want to change the base?
Conversation
…niSubComponent. Based on the acheetham branch https://github.com/acheetham/videoPlayer/tree/FLUID-4851
…l so that transcripts and captions use different data. Make it work. Filter languages which we get from Amara so that local captions/transcripts will have a higher priority.
…video source. Component always creates itself.
… available languages and uses them.
…and transcripts if nothing got retrieved from Amara.
Updated the known issues section of the README with some more issues that came up during release testing.
…ration test to reflect that. Also removing timeout from the integration test.
Conflicts: js/VideoPlayer_controllers.js
…ality to handle relative paths.
…s to accommodate those changes.
…subtitlesFinder testing.
…toring its tests.
…ubtitlesFinder and its DataSet component.
@@ -0,0 +1,60 @@ | |||
/* | |||
Copyright 2012 OCAD University |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copyright data should be 2013
args: ["{arguments}.0"] | ||
} | ||
}, | ||
model: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why these are in a model, instead of plain default options.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because those values could be changed potentially. There could be the case when the same dataSource would be used to query 2 different webservices. In that case we want to change those values and according to our conventions options block has settings which do not change when state of the component changes
|
NOTE: Alex's pull request before leaving. This pull request needs a code review.
http://issues.fluidproject.org/browse/VP-211