fluid-project/videoPlayer — pull request #12 by acheetham (2012-01-19 17:10:08+00:00)
Title: FLUID-4546 template and controls cleanup

This branch is based off cindyli's FLUID-4545 branch: That branch should be merged first. (If there are significant conflicts with this branch, let me know - I'll probably have an easier time addressing them.)

This branch needs a good code review, including (but not limited to):
- does the factoring of the 'toggleButton' subcomponent make sense?
- do the CSS classes for button styling (i.e. hover/pressed/etc.) make sense? I suspect I've been inconsistent in naming and/or application.
- does the scrubber subcomponent need work? (I didn't touch it)

Notes:

- The showing/hiding of the caption area does not work yet (tests fail). We decided not to worry about this until we re-work the captions using captionator and/or mediaelement.js

- Captions are not loaded in non-HTML5 browsers. Again, we decided to leave this until we re-work the captions using captionator and/or mediaelement.js

- The UI is not made pretty yet; that will be addressed with FLUID-4556. The interactions are in place, and the class swapping is in place, but this branch uses temporary placeholder images for the buttons, and no work has been done on layout, etc.