fluid-work IRC Logs-2013-03-07
[08:20:58 CST(-0600)] <jhung> http://wiki.fluidproject.org/display/fluid/FLOE+2013+Q1+Roadmap
[10:04:41 CST(-0600)] <Justin_o> yzen, cindyli: i've pushed up an eventedModel grade to my FLUID-4927 branch in my github repo
[10:04:41 CST(-0600)] <Justin_o> https://github.com/jobara/infusion/commit/92519d42bffeb03a6c69e29884e32f008e0ead38
[10:04:48 CST(-0600)] <Justin_o> can you take a look and see what you think
[10:05:11 CST(-0600)] <Justin_o> i'm also thinking now that we don't need the onRequestChange event and could go directly through the applier instead… but i'm open to leaving it there
[10:05:33 CST(-0600)] <cindyli> sure, thanks, Justin_o
[10:06:57 CST(-0600)] <alexn> cindyli, michelled: I have a question about VideoPlayer. Is there any specific reason why captions and transcripts reside in options but not in the model? Basically, I'm working on the unisub component which is supposed to query Amara for available languages and then extend the existing list. In this case maybe it is a good idea to move captions and transcripts into the VP model to avoid options modifications.
[10:08:22 CST(-0600)] <cindyli> alexn: you mean the contents of captions and transcripts, right?
[10:09:44 CST(-0600)] <cindyli> or the paths that point to the contents
[10:09:52 CST(-0600)] <alexn> cindyli: objects which have source pointing to the captions, their type, label and language
[10:11:50 CST(-0600)] <yzen> Justin_o: re onRequestChange, I think it makes sense to keep it in case the user does not want to know about the applier
[10:12:15 CST(-0600)] <alexn> cindyli: integrator can specify list of sources for captions and transcripts. And those lists currently reside in the options of the video player
[10:13:01 CST(-0600)] <cindyli> alexn: i think the reason they are in the options rather than model is cuz they are not things that indicate the model state. They are data that don't get changed often, in most cases they only get populated once and remain unchanged. Their behaviors fit better into "options" rather than "model"
[10:15:15 CST(-0600)] <alexn> cindyli: oh I see.
[10:21:31 CST(-0600)] <Justin_o> yzen: made the change you suggested, of using the fluid.makeArray function and the member feature
[10:22:13 CST(-0600)] <yzen> Justin_o: nice
[10:23:46 CST(-0600)] <Justin_o> yzen: thanks.. we should probably talk some point today about where we're going to manage the dependencies between our branches.. seeing as how cindyli may want to use this grade in her code.
[10:23:57 CST(-0600)] <yzen> yep
[10:24:03 CST(-0600)] <alexn> Justin_o, michelled, cindyli: I assume it makes sense for me to move captions and transcripts into the model since those arrays will be modified by unisub component. And looking into the future, we possibly want to check Amara for captions inside time interval during a videoPlayer lifespan on the page. What do you think?
[10:32:56 CST(-0600)] <cindyli> hi, Justin_o, I wonder how to hook up a listener for a model path with your eventedModel component. can i do that?
[10:44:21 CST(-0600)] <cindyli> or it only handles requestChange at this point?
[10:45:20 CST(-0600)] <Justin_o> cindyli: you just pass in a modelPath and then list to the afterModelChanged event