From fbb8a6e7ff360f9c83553d0060be046b49c164a1 Mon Sep 17 00:00:00 2001
From: Techywarrior <techywarrior@gmail.com>
Date: Wed, 27 Mar 2013 12:23:42 -0700
Subject: [PATCH 1/3] hide language, resolution and framerate if not relevant

---
 dashboard-ui/scripts/Itemdetailpage.js | 30 ++++++++++++++++++--------
 1 file changed, 21 insertions(+), 9 deletions(-)

diff --git a/dashboard-ui/scripts/Itemdetailpage.js b/dashboard-ui/scripts/Itemdetailpage.js
index 8c54ac4fac..1381ff6290 100644
--- a/dashboard-ui/scripts/Itemdetailpage.js
+++ b/dashboard-ui/scripts/Itemdetailpage.js
@@ -453,16 +453,18 @@
 
         var page = $.mobile.activePage;
         var html = '';
+        var hasVideo = false;
+        var hasLanguage = false;
 
         html += '<table class="ui-responsive table-stroke" id="mediaInfo-table" data-role="table" data-mode="reflow">';
         html += '<thead><tr>';
         html += '<th data-priority="1">Type</th>';
         html += '<th data-priority="2">Codec</th>';
-        html += '<th data-priority="3">Language</th>';
-        html += '<th data-priority="4">Resolution</th>';
+        html += '<th data-priority="3" class="language">Language</th>';
+        html += '<th data-priority="4" class="resolution">Resolution</th>';
         html += '<th data-priority="6">Channels</th>';
         html += '<th data-priority="7">Bitrate</th>';
-        html += '<th data-priority="8">Framerate</th>';
+        html += '<th data-priority="8" class="framerate">Framerate</th>';
         html += '<th data-priority="9">Flags</th>';
         html += '</tr></thead>';
         html += '<tbody>';
@@ -471,18 +473,23 @@
 
             var stream = item.MediaStreams[i];
 
+            if (stream.Type == "Video") {
+                hasVideo = true;
+            }
+
             html += '<tr>';
             html += '<td>' + stream.Type + '</td>';
             html += '<td>' + stream.Codec +'</td>';
 
-            if (stream.Language)
-                html += '<td>' + stream.Language +'</td>';
-            else html += '<td></td>';
+            if (stream.Language) {
+                hasLanguage = true;
+                html += '<td class="language">' + stream.Language +'</td>';
+            } else html += '<td class="language"></td>';
 
             if (stream.Type == "Video") {
-                html += '<td>'+ stream.Width +'/'+ stream.Height +' ('+ stream.AspectRatio +')</td>';
+                html += '<td class="resolution">'+ stream.Width +'/'+ stream.Height +' ('+ stream.AspectRatio +')</td>';
             }else {
-                html += '<td></td>';
+                html += '<td class="resolution"></td>';
             }
 
             if (stream.Channels) {
@@ -495,7 +502,7 @@
 
             if (stream.Type == "Video") {
                 var framerate = stream.AverageFrameRate || stream.RealFrameRate;
-                html += '<td>'+ framerate +'</td>';
+                html += '<td class="framerate">'+ framerate +'</td>';
             }else {
                 html += '<td></td>';
             }
@@ -513,6 +520,11 @@
         html += '</tbody></table>';
 
         $('#mediaInfoContent', page).html(html).trigger('create');
+        if (!hasLanguage) $('#mediaInfoCollapsible #mediaInfo-table .language', page).hide();
+        if (!hadVideo) {
+            $('#mediaInfoCollapsible #mediaInfo-table .resolution', page).hide();
+            $('#mediaInfoCollapsible #mediaInfo-table .framerate', page).hide();
+        }
         $('#mediaInfoCollapsible', page).show();
     },
 

From d578faefb48133c5f33edbd82a82e2fa84092e05 Mon Sep 17 00:00:00 2001
From: Techywarrior <techywarrior@gmail.com>
Date: Wed, 27 Mar 2013 12:27:00 -0700
Subject: [PATCH 2/3] typo

---
 dashboard-ui/scripts/Itemdetailpage.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dashboard-ui/scripts/Itemdetailpage.js b/dashboard-ui/scripts/Itemdetailpage.js
index 1381ff6290..f6903e5559 100644
--- a/dashboard-ui/scripts/Itemdetailpage.js
+++ b/dashboard-ui/scripts/Itemdetailpage.js
@@ -521,7 +521,7 @@
 
         $('#mediaInfoContent', page).html(html).trigger('create');
         if (!hasLanguage) $('#mediaInfoCollapsible #mediaInfo-table .language', page).hide();
-        if (!hadVideo) {
+        if (!hasVideo) {
             $('#mediaInfoCollapsible #mediaInfo-table .resolution', page).hide();
             $('#mediaInfoCollapsible #mediaInfo-table .framerate', page).hide();
         }

From d95a277bbe0a62787ccbccba3843e213a9c61a7b Mon Sep 17 00:00:00 2001
From: Techywarrior <techywarrior@gmail.com>
Date: Wed, 27 Mar 2013 12:37:33 -0700
Subject: [PATCH 3/3] change name for embedded image in audio stream

---
 dashboard-ui/scripts/Itemdetailpage.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/dashboard-ui/scripts/Itemdetailpage.js b/dashboard-ui/scripts/Itemdetailpage.js
index f6903e5559..cb6fdc6659 100644
--- a/dashboard-ui/scripts/Itemdetailpage.js
+++ b/dashboard-ui/scripts/Itemdetailpage.js
@@ -477,8 +477,11 @@
                 hasVideo = true;
             }
 
+            if (item.MediaType == "Audio" && stream.Type == "Video") var type = "Embedded Image";
+            else var type = stream.Type;
+
             html += '<tr>';
-            html += '<td>' + stream.Type + '</td>';
+            html += '<td>' + type + '</td>';
             html += '<td>' + stream.Codec +'</td>';
 
             if (stream.Language) {