From 65412574a07c7c618657cf3c9f04862c5314a7ed Mon Sep 17 00:00:00 2001
From: Roland Gruber <post@rolandgruber.de>
Date: Thu, 14 Nov 2019 21:56:15 +0100
Subject: [PATCH] styling updates

---
 lam/lib/lists.inc                | 15 +++++++++------
 lam/style/500_layout.css         |  4 ++++
 lam/style/responsive/120_lam.css |  6 +++---
 3 files changed, 16 insertions(+), 9 deletions(-)

diff --git a/lam/lib/lists.inc b/lam/lib/lists.inc
index 44528327..5c294a2a 100644
--- a/lam/lib/lists.inc
+++ b/lam/lib/lists.inc
@@ -424,17 +424,17 @@ class lamList {
 			$buttons = new htmlGroup();
 			if (strtolower($attributeName) == $this->sortColumn) {
 				if ($this->sortDirection < 0) {
-					$buttons->addElement(new htmlLink(null, $link . '&sortdirection=1', '../../graphics/downarrows.png'));
-					$buttons->addElement(new htmlLink(null, $link . '&sortdirection=-1', '../../graphics/uparrows-black.png'));
-				}
-				else {
 					$buttons->addElement(new htmlLink(null, $link . '&sortdirection=1', '../../graphics/downarrows-black.png'));
 					$buttons->addElement(new htmlLink(null, $link . '&sortdirection=-1', '../../graphics/uparrows.png'));
 				}
+				else {
+					$buttons->addElement(new htmlLink(null, $link . '&sortdirection=1', '../../graphics/downarrows.png'));
+					$buttons->addElement(new htmlLink(null, $link . '&sortdirection=-1', '../../graphics/uparrows-black.png'));
+				}
 			}
 			else {
-				$buttons->addElement(new htmlLink(null, $link . '&sortdirection=1', '../../graphics/downarrows.png'));
-				$buttons->addElement(new htmlLink(null, $link . '&sortdirection=-1', '../../graphics/uparrows.png'));
+				$buttons->addElement(new htmlLink(null, $link . '&sortdirection=1', '../../graphics/downarrows-black.png'));
+				$buttons->addElement(new htmlLink(null, $link . '&sortdirection=-1', '../../graphics/uparrows-black.png'));
 			}
 			$sortElements[] = $buttons;
 		}
@@ -836,6 +836,7 @@ class lamList {
 			$suffixSelect->setRightToLeftTextDirection(true);
 			$suffixSelect->setSortElements(false);
 			$suffixSelect->setHasDescriptiveElements(true);
+			$suffixSelect->setCSSClasses(array('margin5', 'lam-listouselection-height'));
 			$group->addElement($suffixSelect);
 			$group->addElement(new htmlSpacer('5px', null));
 		}
@@ -883,11 +884,13 @@ class lamList {
 		$right->add($this->listShowOUSelection(), 12, 12, 10);
 		$rightButtonGroup = new htmlGroup();
 		$refreshButton = new htmlButton('refresh', 'refresh.png', true);
+		$refreshButton->setCSSClasses(array('margin5 lam-listouselection-height'));
 		$refreshButton->setTitle(_("Refresh"));
 		$rightButtonGroup->addElement($refreshButton);
 		$settingsLink = new htmlLink('', '#', '../../graphics/tools.png');
 		$settingsLink->setOnClick('listShowSettingsDialog(\'' . _('Change list settings') . '\', \'' . _('Ok') . '\', \'' . _('Cancel') . '\');');
 		$settingsLink->setTitle(_('Change settings'));
+		$settingsLink->setCSSClasses(array('margin5 lam-listouselection-height'));
 		$rightButtonGroup->addElement($settingsLink);
 		$right->add($rightButtonGroup, 12, 12, 2);
 
diff --git a/lam/style/500_layout.css b/lam/style/500_layout.css
index b92b8a94..32e07b6a 100644
--- a/lam/style/500_layout.css
+++ b/lam/style/500_layout.css
@@ -264,6 +264,10 @@ table.accountlist input[type='text'] {
 	padding-bottom: 0.5rem;
 }
 
+.lam-listouselection-height {
+	height: 27px;
+}
+
 table.collapse {
 	border-collapse:collapse;
 }
diff --git a/lam/style/responsive/120_lam.css b/lam/style/responsive/120_lam.css
index 90cba6a6..190b9c42 100644
--- a/lam/style/responsive/120_lam.css
+++ b/lam/style/responsive/120_lam.css
@@ -112,9 +112,9 @@ table.responsive-table th {
 
 table.responsive-table td {
 	overflow: hidden;
-	vertical-align: top;
-	padding-bottom: 0.3rem;
-	padding-top: 0.3rem;
+	vertical-align: middle;
+	padding-bottom: 0.1rem;
+	padding-top: 0.1rem;
 	padding-right: 0.3rem;
 	padding-left: 0.3rem;
 	word-break: break-all;