Merge branch 'develop' into features/JAL-1793VCF

[jalview.git] / src / jalview / datamodel / SequenceFeature.java

diff --git a/src/jalview/datamodel/SequenceFeature.java b/src/jalview/datamodel/SequenceFeature.java
index f5a9b42..420ade1 100755 (executable)
--- a/src/jalview/datamodel/SequenceFeature.java
+++ b/src/jalview/datamodel/SequenceFeature.java
@@ -62,6 +62,8 @@ public class SequenceFeature implements FeatureLocationI
   static
   {
     INFO_KEYS.put("CSQ", ",");
+    // todo capture second level metadata (CSQ FORMAT)
+    // and delimiter "|" so as to report in a table within a table?
   }
 
   /*
@@ -586,23 +588,23 @@ public class SequenceFeature implements FeatureLocationI
         {
           continue; // to avoid double reporting
         }
-        sb.append("<tr><td>").append(key).append("</td><td>");
         if (INFO_KEYS.containsKey(key))
         {
           /*
            * split selected INFO data by delimiter over multiple lines
            */
-          sb.append("</td></tr>");
           String delimiter = INFO_KEYS.get(key);
           String[] values = entry.getValue().toString().split(delimiter);
           for (String value : values)
           {
-            sb.append("<tr><td>&nbsp</td><td>").append(value)
+            sb.append("<tr><td>").append(key).append("</td><td>")
+                    .append(value)
                     .append("</td></tr>");
           }
         }
         else
-        {
+        { // tried <td title="key"> but it failed to provide a tooltip :-(
+          sb.append("<tr><td>").append(key).append("</td><td>");
           sb.append(entry.getValue().toString()).append("</td></tr>");
         }
       }