From: Matti Tahvonen <matti.tahvonen@itmill.com>
Date: Fri, 13 Jul 2007 14:04:32 +0000 (+0000)
Subject: Better support for Table with widgets inside. (no row selects on widget clicks)
X-Git-Tag: 6.7.0.beta1~6172
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=a0bab0d939035fcccdc1cb3c8840e734316425fc;p=vaadin-framework.git

Better support for Table with widgets inside. (no row selects on widget clicks)

svn changeset:1857/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 206c4a82a1..7a5b0a26e8 100644
--- a/src/com/itmill/toolkit/terminal/gwt/client/ui/IScrollTable.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ui/IScrollTable.java
@@ -1413,7 +1413,7 @@ public class IScrollTable extends Composite implements Paintable, ITable, Scroll
 
 		}
 
-		public class IScrollTableRow extends Panel  implements IActionOwner {
+		public class IScrollTableRow extends Panel implements IActionOwner {
 			
 			Vector childWidgets = new Vector();
 			private boolean selected = false;
@@ -1512,13 +1512,19 @@ public class IScrollTable extends Composite implements Paintable, ITable, Scroll
 				return false;
 			}
 
+			/*
+			 * React on click that occur on content cells only
+			 */
 			public void onBrowserEvent(Event event) {
+				String s = DOM.getAttribute(DOM.eventGetTarget(event), "className");
 				switch (DOM.eventGetType(event)) {
 				case Event.ONCLICK:
-					client.console.log("Row click");
-					if(selectMode > ITable.SELECT_MODE_NONE) {
-						toggleSelection();
-						client.updateVariable(paintableId, "selected", selectedRowKeys.toArray(), immediate);
+					if("iscrolltable-cellContent".equals(s)) {
+						client.console.log("Row click");
+						if(selectMode > ITable.SELECT_MODE_NONE) {
+							toggleSelection();
+							client.updateVariable(paintableId, "selected", selectedRowKeys.toArray(), immediate);
+						}
 					}
 					break;