Fixes #6932 "Table rows shift a couple of pixels down when selecting a row in IE7"

* Added negative margins to the cell content when the row is focused
* Removed padding/line-height changes from all core themes that previously handled the focus state offset change

svn changeset:18925/svn branch:6.6

diff --git a/WebContent/VAADIN/themes/base/table/table.css b/WebContent/VAADIN/themes/base/table/table.css
index 7308cc4a46caff6b45ed9d1ffeb078074008fd9f..425f8243d6ee0e01706cc87ddb565da80a7ce6cc 100644 (file)
--- a/WebContent/VAADIN/themes/base/table/table.css
+++ b/WebContent/VAADIN/themes/base/table/table.css
@@ -220,6 +220,10 @@
        border-top: 1px dotted #0066bd;
        border-bottom: 1px dotted #0066bd;
 }
+.v-table-focus .v-table-cell-wrapper {
+       margin-top: -1px;
+       margin-bottom: -1px;
+}
 /* row in column selector */\r
 .v-on {
        \r

diff --git a/WebContent/VAADIN/themes/liferay/table/table.css b/WebContent/VAADIN/themes/liferay/table/table.css
index 0991af85562af0cdb93b8a1fb0d1d0da4df68fbc..de9b4ff426c224eaba64b86681dabfc119e505d3 100644 (file)
--- a/WebContent/VAADIN/themes/liferay/table/table.css
+++ b/WebContent/VAADIN/themes/liferay/table/table.css
@@ -103,11 +103,6 @@
        padding-bottom: 3px;
 }
 
-.v-table-focus .v-table-cell-wrapper {
-       padding-top: 2px;
-       padding-bottom: 2px;
-}
-
 .v-table-cell-content {
        border-right: 1px solid #999;
        padding: 0 6px;

diff --git a/WebContent/VAADIN/themes/reindeer/table/table.css b/WebContent/VAADIN/themes/reindeer/table/table.css
index 903e18f6a8295b4b47e5db7b7be2b5423cbd70fc..8ac40ba1163316dd887591b4f5ceb2f970f7f633 100644 (file)
--- a/WebContent/VAADIN/themes/reindeer/table/table.css
+++ b/WebContent/VAADIN/themes/reindeer/table/table.css
@@ -114,10 +114,6 @@
 .v-table .v-selected .v-table-cell-content {
        border-right-color: #466c90;
 }
-.v-table-focus .v-table-cell-wrapper {
-       padding-top: 2px;
-       padding-bottom: 2px;    
-}
 .v-table-column-selector {
        width: 16px;
        height: 20px;

diff --git a/WebContent/VAADIN/themes/runo/table/table.css b/WebContent/VAADIN/themes/runo/table/table.css
index 9fdb83d82f5456b63eed1af1c18ef2c677bedd8b..7a7de1962b0ede053ba17168b1f4d907d07eb9ed 100644 (file)
--- a/WebContent/VAADIN/themes/runo/table/table.css
+++ b/WebContent/VAADIN/themes/runo/table/table.css
@@ -82,9 +82,6 @@ tr.v-table-row-odd:hover {
        /* Do not specify horizontal padding here */
        padding: 3px 0 0 0;
 }
-.v-table-focus .v-table-cell-wrapper {
-       line-height: 21px;
-}
 /* Fix textfield size to correspond label size inside table */
 .v-table-cell-content .v-textfield,
 .v-table-cell-content .v-button {