VScrollTable: If multiple rows selected and drag starts on selected row, client side now hints that all those rows are dragged.

svn changeset:11700/svn branch:6.3

3 files changed, 22 insertions, 12 deletions

diff --git a/WebContent/VAADIN/themes/base/common/common.css b/WebContent/VAADIN/themes/base/common/common.css
index 336fe828a2..d14eb2fa8d 100644
--- a/WebContent/VAADIN/themes/base/common/common.css
+++ b/WebContent/VAADIN/themes/base/common/common.css
@@ -225,9 +225,8 @@ div.v-app-loading {
 .v-drag-element {
 	z-index:60000;
 	position:absolute;
-    opacity: 0.4;
-    filter: alpha(opacity=40);
-    border: 1px solid #999;
+    opacity: 0.5;
+    filter: alpha(opacity=50);
 }
 
 .v-ddwrapper {
diff --git a/WebContent/VAADIN/themes/base/styles.css b/WebContent/VAADIN/themes/base/styles.css
index ae67ae2a7a..34dcf7e570 100644
--- a/WebContent/VAADIN/themes/base/styles.css
+++ b/WebContent/VAADIN/themes/base/styles.css
@@ -1,5 +1,5 @@
-.v-theme-version:after {content:"6_3_0_dev-20100305";}
-.v-theme-version-6_3_0_dev-20100305 {display: none;}
+.v-theme-version:after {content:"6_3_0_dev-20100309";}
+.v-theme-version-6_3_0_dev-20100309 {display: none;}
 /* Automatically compiled css file from subdirectories. */
 
 .v-absolutelayout-wrapper {
@@ -455,9 +455,8 @@ div.v-app-loading {
 .v-drag-element {
 	z-index:60000;
 	position:absolute;
-    opacity: 0.4;
-    filter: alpha(opacity=40);
-    border: 1px solid #999;
+    opacity: 0.5;
+    filter: alpha(opacity=50);
 }
 
 .v-ddwrapper {
@@ -1429,17 +1428,23 @@ div.v-progressindicator-indeterminate-disabled {
 	color: #ddd;
 }
 
-/* TODO remove important, replace with better anchors or order */
 .v-table-row-drag-middle .v-table-cell-content {
 	background-color: cyan;
 }
-/* TODO consider using height neutral styling (scrollbar issues when pagelength = table.length ) */
 .v-table-row-drag-top .v-table-cell-content {
 	border-top: 1px solid cyan ;
 }
+.v-table-row-drag-top .v-table-cell-wrapper {
+	margin-top: -1px; /* compensate the space consumed by border hint */
+}
 .v-table-row-drag-bottom .v-table-cell-content {
 	border-bottom: 1px solid cyan;
 }
+.v-table-row-drag-bottom .v-table-cell-wrapper {
+	margin-bottom: -1px; /* compensate the space consumed by border hint */
+}
+
+
 
 
 .v-tabsheet,
diff --git a/WebContent/VAADIN/themes/base/table/table.css b/WebContent/VAADIN/themes/base/table/table.css
index e26d3aa22c..783fec6604 100644
--- a/WebContent/VAADIN/themes/base/table/table.css
+++ b/WebContent/VAADIN/themes/base/table/table.css
@@ -160,15 +160,21 @@
 	color: #ddd;
 }
 
-/* TODO remove important, replace with better anchors or order */
 .v-table-row-drag-middle .v-table-cell-content {
 	background-color: cyan;
 }
-/* TODO consider using height neutral styling (scrollbar issues when pagelength = table.length ) */
 .v-table-row-drag-top .v-table-cell-content {
 	border-top: 1px solid cyan ;
 }
+.v-table-row-drag-top .v-table-cell-wrapper {
+	margin-top: -1px; /* compensate the space consumed by border hint */
+}
 .v-table-row-drag-bottom .v-table-cell-content {
 	border-bottom: 1px solid cyan;
 }
+.v-table-row-drag-bottom .v-table-cell-wrapper {
+	margin-bottom: -1px; /* compensate the space consumed by border hint */
+}
+
+