From 53d49c223ea04b634404760601bf83ff26d91c43 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Javi=20Mart=C3=ADn?= <javim@elretirao.net>
Date: Fri, 2 Jul 2021 18:33:15 +0200
Subject: [PATCH] Fix font icon line-height

We forgot to include this property when replacing our use of `%fa-icon`,
and it was causing the admin menu to have a blank space at the bottom.

So we're including it again to make sure nothing else breaks because of
this omition.
---
 app/assets/stylesheets/application.scss  | 2 ++
 app/assets/stylesheets/mixins/icons.scss | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/assets/stylesheets/application.scss b/app/assets/stylesheets/application.scss
index 569d50b07..fe57dab83 100644
--- a/app/assets/stylesheets/application.scss
+++ b/app/assets/stylesheets/application.scss
@@ -5,6 +5,8 @@
 @import "font-awesome-sprockets";
 @import "font-awesome/variables";
 @import "font-awesome/path";
+@import "font-awesome/mixins";
+@import "font-awesome/core";
 @import "jquery-ui/autocomplete";
 @import "jquery-ui/datepicker";
 @import "jquery-ui/sortable";
diff --git a/app/assets/stylesheets/mixins/icons.scss b/app/assets/stylesheets/mixins/icons.scss
index 4c2d2bd83..4eb1b09e0 100644
--- a/app/assets/stylesheets/mixins/icons.scss
+++ b/app/assets/stylesheets/mixins/icons.scss
@@ -1,5 +1,5 @@
 %font-icon {
-  display: inline-block;
+  @extend %fa-icon;
   font-family: "Font Awesome 5 Free";
   vertical-align: middle;
 }