"extra" functionality in the fullscreen function

Li, Cindy cli at ocadu.ca
Tue Feb 28 14:26:45 UTC 2012


Hi Alex,

Thanks for trying around. My comments inline:

> I was trying to apply CSS styling as well as change the videoPlayer HTML markup. Since transcripts should obviously go inside into the flc-videoPlayer-video-controller-area which should eliminate any dynamic css height changes and more.
> But this problem is much more complex as I thought since it requires so many CSS changes in every div which is almost like playing Jenga when everything just collapses with one single tiny change.

I'm not convinced that "transcripts should obviously go inside into the flc-videoPlayer-video-controller-area" since the controller area is programmatically controlled to have the same width as video-area, where the transcript-area should be excluded. 

> I put the styling functions into video.bind("loadedmetadata", function () {
> There were initial set for the transcript height. Plus it should eliminate a chance that we are time dependent since at that point we suppose to have all measures based on the loaded video.

"loadedmetadata" event seems a good place for resizing as this event fires once for each movie that loads. 

One suggestion is to define a resize method like "fluid.videoPlayer.resize" and register it as an invoker so other functions/events can call it when resizing is needed.

Thanks.

Cindy


More information about the fluid-work mailing list