From 17a6161d20851a44453434c05892b12a50647fe1 Mon Sep 17 00:00:00 2001
From: nfreear <nfreear>
Date: Mon, 15 Oct 2007 16:59:17 +0000
Subject: [PATCH] Follow up for MDL-11430, Flickr & Youtube tag blocks should
 use list markup - Fix styles.

---
 blocks/tag_youtube/block_tag_youtube.php |  8 ++++----
 theme/standard/styles_layout.css         | 11 ++++++-----
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/blocks/tag_youtube/block_tag_youtube.php b/blocks/tag_youtube/block_tag_youtube.php
index 91ae2bdc9336e..03cc2dcf86517 100644
--- a/blocks/tag_youtube/block_tag_youtube.php
+++ b/blocks/tag_youtube/block_tag_youtube.php
@@ -156,13 +156,13 @@ function render_video_list($xmlobj){
         foreach($videos as $video){
             $text .= '<li>';
             $text .= '<a href="'. s($video['url']) . '">';
-            $text .= '<img alt="" class="youtube-thumb" style="padding:3px;" src="'. $video['thumbnail_url'] .'" /> <span>';
+            $text .= '<img alt="" class="youtube-thumb" src="'. $video['thumbnail_url'] .'" /> <span>';
             $text .= s($video['title']). '</span></a>';
-            $text .= '<br/>';
+            $text .= '<div>';
             $text .= format_time($video['length_seconds']);
-            $text .= "</li>\n";
+            $text .= "</div></li>\n";
         }
-        $text .= "</ul>\n";
+        $text .= "</ul><div class=\"clearer\"></div>\n";
 
         return $text;
     }
diff --git a/theme/standard/styles_layout.css b/theme/standard/styles_layout.css
index 5229764454bb3..08ea60e373d76 100644
--- a/theme/standard/styles_layout.css
+++ b/theme/standard/styles_layout.css
@@ -1398,12 +1398,13 @@ a.skip-block:focus, a.skip-block:active {
 }
 
 .block_tag_youtube .youtube-thumb {
- width:75px;
- height:50px;
+  padding: 3px;
+  padding-bottom: 0.5em;
+  display: block;
+  float: left;
 }
-.block_tag_youtube .yt-video-entry{
- font-size:10px;
- line-height:11px;
+.block_tag_youtube .yt-video-entry li {
+  clear: left;
 }
 
 .block_tag_flickr .flickr-photos {