From 1f705968827f9dea3ee66e5ea9d35f00d360ad9a Mon Sep 17 00:00:00 2001
From: cmcnulty <cmcnulty@kznf.com>
Date: Sat, 09 Nov 2013 21:49:23 -0500
Subject: [PATCH] CSS only solution to focused messages
---
program/steps/mail/func.inc | 11 ++---------
skins/larry/styles.css | 12 ++++++------
skins/larry/mail.css | 5 -----
3 files changed, 8 insertions(+), 20 deletions(-)
diff --git a/program/steps/mail/func.inc b/program/steps/mail/func.inc
index 6317e32..9358848 100644
--- a/program/steps/mail/func.inc
+++ b/program/steps/mail/func.inc
@@ -262,9 +262,6 @@
if (!in_array('threads', $a_show_cols))
array_unshift($a_show_cols, 'threads');
- // always put focus first
- array_unshift($a_show_cols, 'focus');
-
$_SESSION['skin_path'] = $CONFIG['skin_path'];
// set client env
@@ -314,9 +311,6 @@
if (!in_array('threads', $a_show_cols))
array_unshift($a_show_cols, 'threads');
- // always put focus first
- array_unshift($a_show_cols, 'focus');
-
$_SESSION['list_attrib']['columns'] = $a_show_cols;
// Make sure there are no duplicated columns (#1486999)
@@ -342,8 +336,8 @@
if (empty($a_headers))
return;
- // remove 'threads', 'attachment', 'flag', 'status', 'focus' columns, we don't need them here
- foreach (array('threads', 'attachment', 'flag', 'status', 'priority', 'focus') as $col) {
+ // remove 'threads', 'attachment', 'flag', 'status' columns, we don't need them here
+ foreach (array('threads', 'attachment', 'flag', 'status', 'priority') as $col) {
if (($key = array_search($col, $a_show_cols)) !== FALSE)
unset($a_show_cols[$key]);
}
@@ -471,7 +465,6 @@
case 'attachment':
case 'priority':
case 'status':
- case 'focus':
$col_name = '<span class="' . $col .'"> </span>';
break;
case 'threads':
diff --git a/skins/larry/mail.css b/skins/larry/mail.css
index 0c61664..f3b95b8 100644
--- a/skins/larry/mail.css
+++ b/skins/larry/mail.css
@@ -453,11 +453,6 @@
/*** 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 1726370..f0d5ddd 100644
--- a/skins/larry/styles.css
+++ b/skins/larry/styles.css
@@ -1291,18 +1291,18 @@
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;
}
-.records-table td.focus {
- width: 2px;
-}
-.records-table tr.selected > td.focus {
- filter: none;
- background: white;
+/* 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, .records-table tr.focused > td.focus {
--
Gitblit v1.9.1