[jboss-svn-commits] JBL Code SVN: r35865 - labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/table.

[jboss-svn-commits at lists.jboss.org]

Author: ge0ffrey
Date: 2010-11-04 10:40:29 -0400 (Thu, 04 Nov 2010)
New Revision: 35865

Modified:
   labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/table/SortableHeader.java
Log:
sorting icon positioning

Modified: labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/table/SortableHeader.java
===================================================================
--- labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/table/SortableHeader.java	2010-11-04 14:26:41 UTC (rev 35864)
+++ labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/table/SortableHeader.java	2010-11-04 14:40:29 UTC (rev 35865)
@@ -23,7 +23,7 @@
 
     private static String makeImage(ImageResource resource) {
         AbstractImagePrototype prototype = AbstractImagePrototype.create(resource);
-        return prototype.getHTML().replace("style='", "style='position:absolute;right:0px;top:0px;");
+        return prototype.getHTML();
     }
 
     private final SortableHeaderGroup sortableHeaderGroup;
@@ -73,10 +73,14 @@
 
     @Override
     public void render(SafeHtmlBuilder sb) {
-        sb.appendHtmlConstant("<div style='position:relative;cursor:hand;cursor:pointer;padding-right:16px;'>");
+        sb.appendHtmlConstant("<div style='position: relative; cursor: pointer; padding: 0px;'>");
+        sb.appendHtmlConstant("<span style='padding-right: 10px'>");
+        sb.appendEscaped(text);
+        sb.appendHtmlConstant("</span>");
+//        sb.appendHtmlConstant("<div style='position:absolute;right:0px;top:0px;'></div>");
         switch (sortDirection) {
             case NONE:
-                sb.appendHtmlConstant("<div style='position:absolute;display:none;'></div>");
+                // nothing
                 break;
             case ASCENDING:
                 sb.appendHtmlConstant(sortIndex == 0 ? UP_ARROW : SMALL_UP_ARROW);
@@ -87,9 +91,9 @@
             default:
                 throw new IllegalArgumentException("Unknown sortDirection (" + sortDirection + ").");
         }
-        sb.appendHtmlConstant("<div>");
-        sb.appendEscaped(text);
-        sb.appendHtmlConstant("</div></div>");
+        sb.appendHtmlConstant("</div>");
+//        sb.appendHtmlConstant("<div>");
+//        sb.appendHtmlConstant("</div></div>");
     }
 
 }