From fa4d39f7c63e9ff1d4b5d91b94300b9cfae20757 Mon Sep 17 00:00:00 2001
From: Florian Strohmaier <florian.strohmaier@icinga.com>
Date: Mon, 1 Jul 2019 17:08:02 +0200
Subject: [PATCH] CSS: Compensate select line height inconsistencies

---
 public/css/icinga/controls.less | 12 +++++++++---
 public/css/icinga/forms.less    |  3 +++
 2 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/public/css/icinga/controls.less b/public/css/icinga/controls.less
index 00f83a74d..76daf405d 100644
--- a/public/css/icinga/controls.less
+++ b/public/css/icinga/controls.less
@@ -102,12 +102,18 @@ input.search {
   float: left;
 
   li {
+    line-height: 1;
+
     &.active {
+      border-bottom: 2px solid @icinga-blue;
+
       > a,
       > a:hover {
-        border-bottom: 2px solid @icinga-blue;
-        color: @icinga-blue
+        color: @icinga-blue;
+        /* Compensate border-bottom: 2px */
+        margin-bottom: -2px;
       }
+
       > a:hover {
         background: none;
         cursor: default;
@@ -122,7 +128,7 @@ input.search {
 
     > a,
     > span {
-      padding: 0 0.5em 0.25em 0.5em;
+      padding: 0.5em;
     }
     > a:hover {
       background-color: @gray-lighter;
diff --git a/public/css/icinga/forms.less b/public/css/icinga/forms.less
index 2d7be994e..3f0f937d9 100644
--- a/public/css/icinga/forms.less
+++ b/public/css/icinga/forms.less
@@ -164,6 +164,9 @@ form input {
 
 form select {
   padding-right: 1.5625em;
+  /* compensate inconsistent select height calculations */
+  line-height: 1em;
+  height: 2.25em;
 }
 
 /* Remove native dropdown arrow in IE10+ */