git://source.jalview.org / jalview.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
merge from develop
[jalview.git] / src / jalview / gui / JvSwingUtils.java
diff --git a/src/jalview/gui/JvSwingUtils.java b/src/jalview/gui/JvSwingUtils.java
index b6f85b3..2cba586 100644 (file)
--- a/src/jalview/gui/JvSwingUtils.java
+++ b/src/jalview/gui/JvSwingUtils.java
@@ -20,12 +20,15 @@
  */
 package jalview.gui;
 
+import jalview.util.MessageManager;
+
 import java.awt.BorderLayout;
 import java.awt.Color;
 import java.awt.Font;
 import java.awt.GridLayout;
 import java.awt.Rectangle;
 import java.awt.event.ActionListener;
+import java.util.Objects;
 
 import javax.swing.AbstractButton;
 import javax.swing.JButton;
@@ -37,8 +40,6 @@ import javax.swing.JPanel;
 import javax.swing.JScrollBar;
 import javax.swing.SwingConstants;
 
-import jalview.util.MessageManager;
-
 /**
  * useful functions for building Swing GUIs
  * 
@@ -48,27 +49,28 @@ import jalview.util.MessageManager;
 public final class JvSwingUtils
 {
   /**
-   * wrap a bare html safe string to around 60 characters per line using a
-   * <table width=350>
-   * <tr>
-   * <td></td> field
- * @param enclose TODO
- * @param ttext
+   * wrap a bare html safe string to around 60 characters per line using a CSS
+   * style class specifying word-wrap and break-word
+   * 
+   * @param enclose
+   *          if true, add &lt;html&gt; wrapper tags
+   * @param ttext
    * 
    * @return
    */
   public static String wrapTooltip(boolean enclose, String ttext)
   {
+    Objects.requireNonNull(ttext, "Tootip text to format must not be null!");
     ttext = ttext.trim();
-    boolean maxLenghtExceeded = false;
+    boolean maxLengthExceeded = false;
 
     if (ttext.contains("<br>"))
     {
       String[] htmllines = ttext.split("<br>");
       for (String line : htmllines)
       {
-        maxLenghtExceeded = line.length() > 60;
-        if (maxLenghtExceeded)
+        maxLengthExceeded = line.length() > 60;
+        if (maxLengthExceeded)
         {
           break;
         }
@@ -76,19 +78,18 @@ public final class JvSwingUtils
     }
     else
     {
-      maxLenghtExceeded = ttext.length() > 60;
+      maxLengthExceeded = ttext.length() > 60;
     }
 
-    if (!maxLenghtExceeded)
+    if (!maxLengthExceeded)
     {
       return enclose ? "<html>" + ttext + "</html>" : ttext;
     }
-    else
-    {
-      return (enclose ? "<html><table width=350 border=0><tr><td align=justify>"
-              : "")
-              + ttext + ((enclose ? "</td></tr></table></html>" : ""));
-    }
+
+      return (enclose ? "<html>" : "")
+              + "<style> p.ttip {width: 350; text-align: justify; word-wrap: break-word;}</style><p class=\"ttip\">"
+              + ttext + "</p>" + ((enclose ? "</html>" : ""));
+
   }
 
   public static JButton makeButton(String label, String tooltip,
@@ -148,12 +149,8 @@ public final class JvSwingUtils
     JPanel laypanel = new JPanel(new GridLayout(1, 2));
     JPanel labPanel = new JPanel(new BorderLayout());
     JPanel valPanel = new JPanel();
-    // laypanel.setSize(panel.getPreferredSize());
-    // laypanel.setLayout(null);
     labPanel.setBounds(new Rectangle(7, 7, 158, 23));
     valPanel.setBounds(new Rectangle(172, 7, 270, 23));
-    // labPanel.setLayout(new GridLayout(1,1));
-    // valPanel.setLayout(new GridLayout(1,1));
     labPanel.add(label, BorderLayout.WEST);
     valPanel.add(valBox);
     laypanel.add(labPanel);