From d9a37011c32e1313f2b7703733ac595ef208c6a2 Mon Sep 17 00:00:00 2001
From: Chocobozzz <me@florianbigard.com>
Date: Mon, 20 Feb 2023 11:27:49 +0100
Subject: [PATCH] Remove button height

Prefer using padding instead
---
 client/src/sass/bootstrap.scss          | 3 ++-
 client/src/sass/include/_mixins.scss    | 6 ++----
 client/src/sass/include/_variables.scss | 1 -
 3 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/client/src/sass/bootstrap.scss b/client/src/sass/bootstrap.scss
index 7843d3be7..f3e7880ac 100644
--- a/client/src/sass/bootstrap.scss
+++ b/client/src/sass/bootstrap.scss
@@ -339,7 +339,8 @@
 .input-group {
   > .btn,
   > .input-group-text {
-    height: $button-height;
+    padding-top: 4px;
+    padding-bottom: 4px;
   }
 
   > .input-group-text {
diff --git a/client/src/sass/include/_mixins.scss b/client/src/sass/include/_mixins.scss
index 0df849167..34af9e8c1 100644
--- a/client/src/sass/include/_mixins.scss
+++ b/client/src/sass/include/_mixins.scss
@@ -90,9 +90,8 @@
 }
 
 @mixin peertube-input-text($width) {
-  padding: 0 15px;
+  padding: 4px 15px;
   display: inline-block;
-  height: $button-height;
   width: $width;
   max-width: $width;
   color: pvar(--inputForegroundColor);
@@ -401,12 +400,11 @@
   }
 
   select {
-    padding: 0 35px 0 12px;
+    padding: 4px 35px 4px 12px;
     position: relative;
     border: 1px solid pvar(--inputBorderColor);
     background: transparent none;
     appearance: none;
-    height: $button-height;
     text-overflow: ellipsis;
     color: pvar(--mainForegroundColor);
     font-size: $form-input-font-size;
diff --git a/client/src/sass/include/_variables.scss b/client/src/sass/include/_variables.scss
index f16145ae0..8358270da 100644
--- a/client/src/sass/include/_variables.scss
+++ b/client/src/sass/include/_variables.scss
@@ -39,7 +39,6 @@ $expanded-horizontal-margins: 150px;
 $not-expanded-horizontal-margins: 30px;
 
 $button-font-size: 15px;
-$button-height: 30px;
 
 $header-height: 50px;
 $header-border-color: #e9eff6;