Merge pull request #153 from cmcnulty/row-focus

Indicator for row focus, resolves #1489392

5 files changed, 49 insertions, 6 deletions

diff --git a/skins/classic/addressbook.css b/skins/classic/addressbook.css
index 9575ae25f..e4e88ca4c 100644
--- a/skins/classic/addressbook.css
+++ b/skins/classic/addressbook.css
@@ -224,6 +224,19 @@
   -o-text-overflow: ellipsis;
 }
 
+/* This padding-left should be equal to the focused border-left + the focused padding-left */
+#contacts-table thead tr td:first-child,
+#contacts-table tbody tr td:first-child {
+	border-left: 0;
+	padding-left: 8px;
+}
+
+/* because of border-collapse, we make the left border twice what we want it to be - half will be hidden to the left */
+#contacts-table tbody tr.focused > td:first-child {
+	border-left: 4px solid #EBEBEB;
+	padding-left: 4px;
+}
+
 #contacts-table .contact.readonly td
 {
   font-style: italic;
diff --git a/skins/classic/mail.css b/skins/classic/mail.css
index 4f145368d..2b09ba269 100644
--- a/skins/classic/mail.css
+++ b/skins/classic/mail.css
@@ -958,6 +958,20 @@ table.messagelist.fixedcopy
   color: #CC0000;
 }
 
+/* This padding-left minus the focused padding left should be half of the focused border-left */
+.messagelist thead tr td:first-child,
+.messagelist tbody tr td:first-child {
+	border-left: 0;
+	padding-left: 6px;
+}
+
+
+/* because of border-collapse, we make the left border twice what we want it to be - half will be hidden to the left */
+.messagelist tbody tr.focused > td:first-child {
+	border-left: 8px solid #EBEBEB;
+	padding-left: 2px;
+}
+
 .messagelist tr.selected td
 {
   color: #FFFFFF;
diff --git a/skins/larry/addressbook.css b/skins/larry/addressbook.css
index cf4d22d2e..8681cda70 100644
--- a/skins/larry/addressbook.css
+++ b/skins/larry/addressbook.css
@@ -159,6 +159,19 @@
 	font-weight: bold;
 }
 
+/* This padding-left should be equal to the focused border-left + the focused padding-left */
+#contacts-table thead tr td:first-child,
+#contacts-table tbody tr td:first-child {
+	border-left: 0;
+}
+
+/* because of border-collapse, we make the left border twice what we want it to be - half will be hidden to the left */
+#contacts-table tbody tr.focused > td:first-child {
+	border-left: 2px ridge #ffffff;
+	padding-left: 34px;
+	background-position-x: 4px !important; /* important to override selected class */	
+}
+
 #contacts-table .group td.name {
 	background-position: 6px -1555px;
 }
diff --git a/skins/larry/mail.css b/skins/larry/mail.css
index 801c80233..90b3511d4 100644
--- a/skins/larry/mail.css
+++ b/skins/larry/mail.css
@@ -446,11 +446,6 @@ a.iconbutton.threadmode.selected {
 
 /*** message list ***/
 
-/* this is necessary to make FF3 display borders */
-body:-moz-last-node #messagelist {
-	border-collapse: separate;
-}
-
 table.messagelist {
 	z-index: 1;
 }
diff --git a/skins/larry/styles.css b/skins/larry/styles.css
index 23bcd2aeb..810299735 100644
--- a/skins/larry/styles.css
+++ b/skins/larry/styles.css
@@ -1378,9 +1378,18 @@ table.records-table {
 	background-color: #fff;
 }
 
+/* This padding-left minus the focused padding left should be half of the focused border-left */
 .records-table thead tr td:first-child,
 .records-table tbody tr td:first-child {
 	border-left: 0;
+	padding-left: 6px;
+}
+
+
+/* because of border-collapse, we make the left border twice what we want it to be - half will be hidden to the left */
+.records-table tbody tr.focused > td:first-child {
+	border-left: 8px solid #DDDDDD;
+	padding-left: 2px;
 }
 
 .records-table tr.selected td {
@@ -1413,7 +1422,6 @@ table.records-table {
 	color: #ccc !important;
 }
 
-
 /*** iFrames ***/
 
 #aboutframe {