From: Marc Englund <marc.englund@itmill.com>
Date: Tue, 11 Mar 2008 09:12:39 +0000 (+0000)
Subject: Table column selector now has both on and off styles, and checkmark for visible colum... 
X-Git-Tag: 6.7.0.beta1~4951
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=42c244e22ca718eadfbff0b98d89ede1f9112f86;p=vaadin-framework.git

Table column selector now has both on and off styles, and checkmark for visible columns. Fixes #1469

svn changeset:4019/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ui/IScrollTable.java b/src/com/itmill/toolkit/terminal/gwt/client/ui/IScrollTable.java
index 7b8563bc7e..111a209edc 100644
--- a/src/com/itmill/toolkit/terminal/gwt/client/ui/IScrollTable.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ui/IScrollTable.java
@@ -1479,11 +1479,12 @@ public class IScrollTable extends Composite implements Table, ScrollListener,
                 final StringBuffer buf = new StringBuffer();
                 if (collapsed) {
                     buf.append("<span class=\"i-off\">");
+                } else {
+                    buf.append("<span class=\"i-on\">");
                 }
                 buf.append(super.getHTML());
-                if (collapsed) {
-                    buf.append("</span>");
-                }
+                buf.append("</span>");
+
                 return buf.toString();
             }
 
diff --git a/src/com/itmill/toolkit/terminal/gwt/public/default/table/table.css b/src/com/itmill/toolkit/terminal/gwt/public/default/table/table.css
index 745a451338..c427ca02e9 100644
--- a/src/com/itmill/toolkit/terminal/gwt/public/default/table/table.css
+++ b/src/com/itmill/toolkit/terminal/gwt/public/default/table/table.css
@@ -160,9 +160,15 @@
 }
 
 
-/* disabled row in column selector */
-.i-off {
-	font-style: italic;
+/* rowe in column selector */
+.i-on div {
+	background-image: url('img/check.gif');
+	background-repeat: no-repeat;
+	background-position: 2px center;
+	text-indent: 6px;
+}
+.i-off div {
+	text-indent: 6px;
 	color:
 }