Permalink
Browse files

FLUID-4839: Minor changes based on code review - uncommenting some co…

…mmented out lines and putting in a comment.
  • Loading branch information...
1 parent bf52850 commit 4a0fee304ec3d1608b9d8043d34092018776fd65 @michelled committed Dec 4, 2012
Showing with 10 additions and 8 deletions.
  1. +8 −8 demos/Mammals.js
  2. +2 −0 js/VideoPlayer.js
View
@@ -152,14 +152,14 @@ https://github.com/fluid-project/infusion/raw/master/Infusion-LICENSE.txt
videoTitle: "Polar Mammal Adaptation",
video: {
sources: [
- // {
- // src: "videos/PolarMammalAdaptations/PolarMammalAdaptations.mp4",
- // type: "video/mp4"
- // },
- // {
- // src: "videos/PolarMammalAdaptations/PolarMammalAdaptations.webm",
- // type: "video/webm"
- // },
+ {
+ src: "videos/PolarMammalAdaptations/PolarMammalAdaptations.mp4",
+ type: "video/mp4"
+ },
+ {
+ src: "videos/PolarMammalAdaptations/PolarMammalAdaptations.webm",
+ type: "video/webm"
+ },
{
src: "http://www.youtube.com/v/3_3p2ylZDAE",
type: "video/youtube"
View
@@ -542,6 +542,8 @@ https://github.com/fluid-project/infusion/raw/master/Infusion-LICENSE.txt
that.refreshView();
var video = that.locate("video");
video.attr("aria-label", that.options.strings.videoTitlePreface + ": " + that.options.videoTitle);
+ // Setting the width and height attributes to respect the CSS API for setting the size of the video
+ // This is required for cross browser sizing of the video
video.attr("width", video.css("width"));
video.attr("height", video.css("height"));

0 comments on commit 4a0fee3

Please sign in to comment.