Fixing Developer edition layout issues

ui/views/newrelic/_table.rhtml

@@ -1,10 +1,12 @@
 <table border="0" cellpadding="5" cellspacing="0">
-	<tr class='table_header'>
-	<th align=left>Metric</th>
-	<th>Count</th>
-	<th colspan=2>Exclusive</th>
-	<th></th>
-	<th colspan=2>Total</th>
+	<thead>
+	<tr>
+	<td>Metric</td>
+	<td>Count</td>
+	<td colspan=2>Exclusive</td>
+	<td/>
+	<td colspan=2>Total</td>
    </tr>
+   </thead>
 	<%= render :partial => 'segment_row', :collection => table %>
 </table>

ui/views/newrelic/index.rhtml

@@ -2,10 +2,12 @@
 	<tr>
 		<td valign=top>
 			<table>
+				<thead>
 				<tr>
-          <th nowrap=true>Resp Time</th>
-					<th>URL</th>
+          			<td>Resp Time</td>
+					<td>URL</td>
 				</tr>
+				</thead>
 				<% @samples.each do |sample| %>
 					<tr>
 					<td align=right nowrap=true>

ui/views/newrelic/stylesheets/style.css

@@ -105,19 +105,17 @@ var{
 	font-style: normal;
 	font-size: 1.1em;
 }
-th.strong
+
+th, td, thead, tfoot
 {
-	font-weight: bold;
+	font-size: .95em;	
 }
 
+/* regular tables use thead instead of th.  th is only used by name/value table layouts */
 th
 {
-	font-size: .95em;
-	text-align:left;
-	font-weight: normal;
-	line-height:100%;
-	background-color: #EEEEEE;
 	padding: 5px 5px 5px 5px;
+	text-align: right;
 }
 /*************************************
 +Seldon Specific - need to preserve
@@ -208,11 +206,18 @@ table.agents
 	width: 100%;
 }
 
-tfoot
+thead
 {
-
+	text-align:left;
+	font-weight: normal;
+	line-height:100%;
 	background-color: #EEEEEE;
-	font-size: .95em; 
+	padding: 5px 5px 5px 5px;
+}
+
+tfoot
+{
+	background-color: #EEEEEE; 
 	color: #333;
 }
 
@@ -249,8 +254,6 @@ td.time_nav
 
 td
 {
-	border: 0px;
-	font-size: .95em;
 	text-align: left;
 	padding: 5px 5px 5px 5px;
 }

ui/views/newrelic/view_sample.rhtml

@@ -1,28 +1,28 @@
 <div id='summary_table'>
 <table align=left width="100%">
   <tr>
-    <th align="right">URL:</th>
+    <th>URL:</th>
     <td><%= @sample.params[:uri]%></td>
   </tr>
   <tr>
-    <th align="right">Controller:</th>
+    <th>Controller:</th>
     <td><%= @sample_controller_name%></td>
   </tr>
   <tr>
-    <th align="right">Action:</th>
+    <th>Action:</th>
     <td><%= @sample_action_name%></td>
   </tr>
   <tr>
-    <th align="right">Timestamp:</th>
+    <th>Timestamp:</th>
     <td><%= format_timestamp(@sample.start_time) %></td>
   </tr>
   <tr>
-    <th align="right">Duration:</th>
+    <th>Duration:</th>
     <td><%= colorize(@sample.duration, 1, 2) %> ms</td>
   </tr>
   <% unless @request_params.empty? %>
     <tr>
-      <th align="right" valign="top">Params:</th>
+      <th valign="top">Params:</th>
       <td>
         <%= link_to_function("#{@request_params.length}...", :id => "params_link") do |page| 
               page[:request_params].show
@@ -32,7 +32,7 @@
           <div id='request_params' style="display: none;">
           <small>
             <% @request_params.each do |k,v| %>
-              <b>&nbsp &nbsp <%= k %>: </b><%=v%><br/>
+              <b>&nbsp; &nbsp; <%= k %>: </b><%=v%><br/>
             <% end %>
           </small>
         </div>
@@ -66,9 +66,7 @@ end
 	<% @pie_chart.width = 800; @pie_chart.height = 300 %>
 	<%= @pie_chart.render %>
 	<br/>
-	<table>
-		<%= render :partial => 'table', :object => @sample.breakdown_data(6) %>
-	</table>
+	<%= render :partial => 'table', :object => @sample.breakdown_data(6) %>
 </div>
 
 <div id='table' align=left style="display: none;">
@@ -84,33 +82,37 @@ end
     so that the rows logger is loaded first.  
   <% else %>
 	<table width=100%>
-		<tr class='table_header'>
-			<th>Timestamp</th>
-			<th>Duration</th>
-			<th></th>
-			<th align=left>SQL</th>
+		<thead>
+		<tr>
+			<td>Timestamp</td>
+			<td>Duration</td>
+			<td></td>
+			<td>SQL</td>
 		</tr>
+		</thead>
 
 		<%= render :partial => 'sql_row', :collection => @sample.sql_segments %>
 	</table>
   <% end %>
 </div>
 
-<div id='text_dump' style="display: none;" style="display: none;">
+<div id='text_dump' style="display: none;">
 <pre>
 <%= @sample.to_s %>
 </pre>
 </div>
 
 <div id='details' align='center' style="display: none;">
 <table>
-  <tr class='table_header'>
-    <th>Metric</th>
-    <th>Timestamp<br/><small>seconds</small></th>
-    <th>Duration<br/><small>ms</small></th>
-    <th>Exclusive<br/><small>ms</small></th>
-    <th></th>
+  <thead>
+  <tr>
+    <td>Metric</td>
+    <td>Timestamp<br/><small>seconds</small></td>
+    <td>Duration<br/><small>ms</small></td>
+    <td>Exclusive<br/><small>ms</small></td>
+    <td></td>
   </tr>
+  </thead>
   <%= render_sample_details @sample %>
 </table>
 </div>