From 729c0bca0989289760b83af35cbcb1d294558690 Mon Sep 17 00:00:00 2001
From: Frances Botsford <frances@edx.org>
Date: Wed, 9 Oct 2013 14:40:01 -0400
Subject: [PATCH] updating video download links to a list - review feedback

---
 .../lib/xmodule/xmodule/css/video/display.scss   |  6 +-----
 lms/templates/video.html                         | 16 ++++++++--------
 2 files changed, 9 insertions(+), 13 deletions(-)

diff --git a/common/lib/xmodule/xmodule/css/video/display.scss b/common/lib/xmodule/xmodule/css/video/display.scss
index 75ce1f6c61..3294549901 100644
--- a/common/lib/xmodule/xmodule/css/video/display.scss
+++ b/common/lib/xmodule/xmodule/css/video/display.scss
@@ -32,13 +32,9 @@ div.video {
     display: inline-block;
     margin: ($baseline*.75) ($baseline/2) 0 0;
 
-    p {
-      display: inline-block;
-      margin: 0;
-    }
-
     a {
       @include transition(all 0.25s ease-in-out 0s);
+      @include font-size(14);
       display: inline-block;
       border-radius: 3px 3px 3px 3px;
       background-color: $white;
diff --git a/lms/templates/video.html b/lms/templates/video.html
index d9770c0977..a4fa42faee 100644
--- a/lms/templates/video.html
+++ b/lms/templates/video.html
@@ -76,19 +76,19 @@
     </div>
 
     <div class="focus_grabber last"></div>
-
+<ul class="downloads-wrapper">
 % if sources.get('main'):
-    <div class="video-sources">
-        <p>${('<a href="%s">' + _('Download video') + '</a>') % sources.get('main')}</p>
-    </div>
+    <li class="video-sources">
+        ${('<a href="%s">' + _('Download video') + '</a>') % sources.get('main')}
+    </li>
 % endif
 
 % if track:
-    <div class="video-tracks">
-        <p>${('<a href="%s">' + _('Download timed transcript') + '</a>') % track}</p>
-    </div>
+    <li class="video-tracks">
+        ${('<a href="%s">' + _('Download timed transcript') + '</a>') % track}
+    </li>
 % endif
-
+</ul>
 </div>