From 6266d5fdcedde07ea092c14c1345ddbe2c35bf11 Mon Sep 17 00:00:00 2001
From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Date: Fri, 25 Sep 2020 12:48:30 +0100
Subject: [PATCH] Bug 26534: (follow-up) Only center when using patronimages

I felt the change to always center the user title worked well when
patronimages was enabled, but not so well when it was not (it looked
strange without a background for non-staff users).
---
 .../prog/css/src/staff-global.scss            |  5 ++-
 .../prog/en/includes/circ-menu.inc            | 33 +++++++++++--------
 2 files changed, 24 insertions(+), 14 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss b/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss
index b5285b7907..8b22608898 100644
--- a/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss
+++ b/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss
@@ -1381,7 +1381,6 @@ dd {
     margin-top: -.5em;
 
     h5 {
-        text-align: center;
         border-right: 1px solid #B9D8D9;
         margin-bottom: 0;
         padding-bottom: .5em;
@@ -1391,6 +1390,10 @@ dd {
         &:empty {
             border-right: 0;
         }
+
+        &.center {
+            text-align: center;
+        }
     }
 
     ul {
diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc
index ec23e4f3b4..ef7e5d0ce0 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc
@@ -5,20 +5,14 @@
 [% SET patrongrade = ( patron.has_permission({ 'catalogue' => 1 }) ) ? 'staffgrade' : 'normalgrade' %]
 <div class="patroninfo [% patrongrade | html %]">
 
-<h5>
-    [% IF ( patrongrade == 'staffgrade' ) %]
-        <i class="fa fa-shield" title="Staff patron"></i>
-    [% END %]
-    [% INCLUDE 'patron-title.inc' %]
-</h5>
-
-[% IF Koha.Preference('showLastPatron') %]
-    <input type="hidden" id="hiddenborrowernumber" value="[% patron.borrowernumber | html %]">
-    <input type="hidden" id="hiddenborrowername" value="[% patron.surname | html %] [% patron.firstname | html %]">
-    <input type="hidden" id="hiddenborrowercard" value="[% patron.cardnumber | html %]">
-[% END %]
-
 [% IF ( patronimages ) %]
+    <h5 class="text-center">
+        [% IF ( patrongrade == 'staffgrade' ) %]
+            <i class="fa fa-shield fa-fw" title="Staff patron"></i>
+        [% END %]
+        [% INCLUDE 'patron-title.inc' %]
+    </h5>
+
     <div>
         <div class="patronimage-container">
             [% IF ( patron.image ) %]
@@ -34,6 +28,19 @@
             [% END %]
         </div>
     </div>
+[% ELSE %]
+    <h5>
+        [% IF ( patrongrade == 'staffgrade' ) %]
+            <i class="fa fa-shield fa-fw" title="Staff patron"></i>
+        [% END %]
+        [% INCLUDE 'patron-title.inc' %]
+    </h5>
+[% END %]
+
+[% IF Koha.Preference('showLastPatron') %]
+    <input type="hidden" id="hiddenborrowernumber" value="[% patron.borrowernumber | html %]">
+    <input type="hidden" id="hiddenborrowername" value="[% patron.surname | html %] [% patron.firstname | html %]">
+    <input type="hidden" id="hiddenborrowercard" value="[% patron.cardnumber | html %]">
 [% END %]
 
 <ul class="patronbriefinfo">
-- 
2.20.1