Got rid of unnecessary 'div' elements in table.

idea/src/org/jetbrains/jet/plugin/highlighter/HtmlTabledDescriptorRenderer.java

@@ -160,7 +160,7 @@ public class HtmlTabledDescriptorRenderer extends TabledDescriptorRenderer {
     private static final DescriptorRenderer.ValueParametersHandler VALUE_PARAMETERS_HANDLER = new DescriptorRenderer.ValueParametersHandler() {
         @Override
         public void appendBeforeValueParameter(@NotNull ValueParameterDescriptor parameter, @NotNull StringBuilder stringBuilder) {
-            stringBuilder.append("<td align=\"right\"><div style=\"white-space:nowrap;font-weight:bold;\">");
+            stringBuilder.append("<td align=\"right\" style=\"white-space:nowrap;font-weight:bold;\">");
         }
 
         @Override
@@ -169,12 +169,12 @@ public class HtmlTabledDescriptorRenderer extends TabledDescriptorRenderer {
             if (!last) {
                 stringBuilder.append(",");
             }
-            stringBuilder.append("</div></td>");
+            stringBuilder.append("</td>");
         }
 
         @Override
         public void appendBeforeValueParameters(@NotNull FunctionDescriptor function, @NotNull StringBuilder stringBuilder) {
-            stringBuilder.append("</div></td>");
+            stringBuilder.append("</td>");
             if (function.getValueParameters().isEmpty()) {
                 tdBold(stringBuilder, "( )");
             }
@@ -188,7 +188,7 @@ public class HtmlTabledDescriptorRenderer extends TabledDescriptorRenderer {
             if (!function.getValueParameters().isEmpty()) {
                 tdBold(stringBuilder, ")");
             }
-            stringBuilder.append("<td><div style=\"white-space:nowrap;font-weight:bold;\">");
+            stringBuilder.append("<td style=\"white-space:nowrap;font-weight:bold;\">");
         }
     };
 
@@ -199,7 +199,7 @@ public class HtmlTabledDescriptorRenderer extends TabledDescriptorRenderer {
             .setTextFormat(DescriptorRenderer.TextFormat.HTML).build();
 
     private static void td(StringBuilder builder, String text) {
-        builder.append("<td><div style=\"white-space:nowrap;\">").append(text).append("</div></td>");
+        builder.append("<td style=\"white-space:nowrap;\">").append(text).append("</td>");
     }
 
     private static void tdSpace(StringBuilder builder) {
@@ -207,19 +207,19 @@ public class HtmlTabledDescriptorRenderer extends TabledDescriptorRenderer {
     }
 
     private static void tdColspan(StringBuilder builder, String text, int colspan) {
-        builder.append("<td colspan=\"").append(colspan).append("\"><div style=\"white-space:nowrap;\">").append(text).append("</div></td>");
+        builder.append("<td colspan=\"").append(colspan).append("\" style=\"white-space:nowrap;\">").append(text).append("</td>");
     }
 
     private static void tdBold(StringBuilder builder, String text) {
-        builder.append("<td><div style=\"white-space:nowrap;font-weight:bold;\">").append(text).append("</div></td>");
+        builder.append("<td style=\"white-space:nowrap;font-weight:bold;\">").append(text).append("</td>");
     }
 
     private static void tdRight(StringBuilder builder, String text) {
-        builder.append("<td align=\"right\"><div style=\"white-space:nowrap;\">").append(text).append("</div></td>");
+        builder.append("<td align=\"right\" style=\"white-space:nowrap;\">").append(text).append("</td>");
     }
 
     private static void tdRightBoldColspan(StringBuilder builder, int colspan, String text) {
-        builder.append("<td align=\"right\" colspan=\"").append(colspan).append("\"><div style=\"white-space:nowrap;font-weight:bold;\">").append(text).append("</div></td>");
+        builder.append("<td align=\"right\" colspan=\"").append(colspan).append("\" style=\"white-space:nowrap;font-weight:bold;\">").append(text).append("</td>");
     }
 
     public static String tableForTypes(String message, String firstDescription, TextElementType firstType, String secondDescription, TextElementType secondType) {

idea/testData/htmlTabledRenderer/diagnostic1.html

@@ -5,165 +5,113 @@ T</b>
 <tr>
 <td width="10%">
 </td>
-<td align="right" colspan="2">
+<td align="right" colspan="2" style="white-space:nowrap;font-weight:bold;">
-<div style="white-space:nowrap;font-weight:bold;">
 <b>
 fun</b>
  &lt;T, E>
- writeToMyList</div>
+ writeToMyList</td>
-</td>
+<td style="white-space:nowrap;font-weight:bold;">
-<td>
+(</td>
-<div style="white-space:nowrap;font-weight:bold;">
+<td align="right" style="white-space:nowrap;font-weight:bold;">
-(</div>
+l : MyList&lt;in T&gt;,</td>
-</td>
+<td align="right" style="white-space:nowrap;font-weight:bold;">
-<td align="right">
+t : T</td>
-<div style="white-space:nowrap;font-weight:bold;">
+<td style="white-space:nowrap;font-weight:bold;">
-l : MyList&lt;in T&gt;,</div>
+)</td>
-</td>
+<td style="white-space:nowrap;font-weight:bold;">
-<td align="right">
-<div style="white-space:nowrap;font-weight:bold;">
-t : T</div>
-</td>
-<td>
-<div style="white-space:nowrap;font-weight:bold;">
-)</div>
-</td>
-<td>
-<div style="white-space:nowrap;font-weight:bold;">
  : Unit <b>
 where</b>
- E : java.lang.Cloneable, E : java.io.Closeable</div>
+ E : java.lang.Cloneable, E : java.io.Closeable</td>
-</td>
 </tr>
 <tr>
-<td colspan="8">
+<td colspan="8" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
+None of the following substitutions</td>
-None of the following substitutions</div>
-</td>
 </tr>
 <tr>
 <td width="10%">
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 (</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <font color=red>
 <b>
 MyList&lt;in jet.Any&gt;</b>
 </font>
 <b>
 ,</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 jet.Any</b>
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 )</b>
-</div>
 </td>
 </tr>
 <tr>
 <td width="10%">
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 (</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 MyList&lt;in jet.Int&gt;</b>
 <b>
 ,</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <font color=red>
 <b>
 jet.Int</b>
 </font>
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 )</b>
-</div>
 </td>
 </tr>
 <tr>
-<td colspan="8">
+<td colspan="8" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
+can be applied to</td>
-can be applied to</div>
-</td>
 </tr>
 <tr>
 <td width="10%">
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 (</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 MyList&lt;jet.Int&gt;</b>
 <b>
 ,</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 jet.Any</b>
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 )</b>
-</div>
 </td>
 </tr>
 </table>
-</html>
+</html>

idea/testData/htmlTabledRenderer/diagnostic2.html

@@ -5,163 +5,111 @@ T</b>
 <tr>
 <td width="10%">
 </td>
-<td align="right" colspan="2">
+<td align="right" colspan="2" style="white-space:nowrap;font-weight:bold;">
-<div style="white-space:nowrap;font-weight:bold;">
 <b>
 ctor</b>
  Cons&lt;T, E : java.lang.Cloneable>
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;font-weight:bold;">
-<div style="white-space:nowrap;font-weight:bold;">
+(</td>
-(</div>
+<td align="right" style="white-space:nowrap;font-weight:bold;">
-</td>
+l : MyList&lt;in T&gt;,</td>
-<td align="right">
+<td align="right" style="white-space:nowrap;font-weight:bold;">
-<div style="white-space:nowrap;font-weight:bold;">
+t : T</td>
-l : MyList&lt;in T&gt;,</div>
+<td style="white-space:nowrap;font-weight:bold;">
-</td>
+)</td>
-<td align="right">
+<td style="white-space:nowrap;font-weight:bold;">
-<div style="white-space:nowrap;font-weight:bold;">
-t : T</div>
-</td>
-<td>
-<div style="white-space:nowrap;font-weight:bold;">
-)</div>
-</td>
-<td>
-<div style="white-space:nowrap;font-weight:bold;">
-</div>
 </td>
 </tr>
 <tr>
-<td colspan="8">
+<td colspan="8" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
+None of the following substitutions</td>
-None of the following substitutions</div>
-</td>
 </tr>
 <tr>
 <td width="10%">
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 (</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <font color=red>
 <b>
 MyList&lt;in jet.Any&gt;</b>
 </font>
 <b>
 ,</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 jet.Any</b>
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 )</b>
-</div>
 </td>
 </tr>
 <tr>
 <td width="10%">
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 (</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 MyList&lt;in jet.Int&gt;</b>
 <b>
 ,</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <font color=red>
 <b>
 jet.Int</b>
 </font>
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 )</b>
-</div>
 </td>
 </tr>
 <tr>
-<td colspan="8">
+<td colspan="8" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
+can be applied to</td>
-can be applied to</div>
-</td>
 </tr>
 <tr>
 <td width="10%">
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 (</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 MyList&lt;jet.Int&gt;</b>
 <b>
 ,</b>
-</div>
 </td>
-<td align="right">
+<td align="right" style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 jet.Any</b>
-</div>
 </td>
-<td>
+<td style="white-space:nowrap;">
-<div style="white-space:nowrap;">
 <b>
 )</b>
-</div>
 </td>
 </tr>
 </table>
-</html>
+</html>