git://source.jalview.org / jalview.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | compact (merge: 72e6064 91ef57a)
merge
authortcofoegbu <tcnofoegbu@dundee.ac.uk>
Thu, 20 Aug 2015 14:00:13 +0000 (15:00 +0100)
committertcofoegbu <tcnofoegbu@dundee.ac.uk>
Thu, 20 Aug 2015 14:00:13 +0000 (15:00 +0100)
1  2 
src/jalview/gui/JvSwingUtils.java patch | diff1 | diff2 | blob | history

diff --combined src/jalview/gui/JvSwingUtils.java
index c4ed0dc,65be457..2cba586
--- 1/src/jalview/gui/JvSwingUtils.java
--- 2/src/jalview/gui/JvSwingUtils.java
+++ b/src/jalview/gui/JvSwingUtils.java
@@@ -49,12 -49,12 +49,12 @@@ import javax.swing.SwingConstants
  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
     */
@@@ -62,15 -62,20 +62,15 @@@
    {
      Objects.requireNonNull(ttext, "Tootip text to format must not be null!");
      ttext = ttext.trim();
-     boolean maxLenghtExceeded = false;
+     boolean maxLengthExceeded = false;
  
 -    /*
 -     * Split into lines if already separated by <br> tags.
 -     * 
 -     * TODO what about </br>, <br/>, <br /> ?
 -     */
      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;
          }
@@@ -78,19 -83,17 +78,18 @@@
      }
      else
      {
-       maxLenghtExceeded = ttext.length() > 60;
+       maxLengthExceeded = ttext.length() > 60;
      }
  
-     if (!maxLenghtExceeded)
+     if (!maxLengthExceeded)
      {
        return enclose ? "<html>" + ttext + "</html>" : ttext;
      }
-     else
-     {
+ 
 -    return enclose ? "<html><style> p.ttip {width: 350; text-align: justify; word-wrap: break-word;}</style><p class=\"ttip\">"
 -            + ttext + "</p></html>"
 -            : ttext;
 +      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,