From 8e6a8972e65758f7a57cb48def15604d7537ed04 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Tue, 25 Apr 2017 16:16:49 +0000 Subject: [PATCH] Bug 18419 - Broken patron-blank image in viewlog.tt Because of a variable name collision the blank patron image doesn't display in viewlog.tt. This patch moves the image from the template to the stylesheet to avoid this problem. This patch also replaces the blank patron image PNG file with an SVG file. SVG support is wide enough to begin using whereever possible. This patch also removes some inline CSS from circ-menu.inc and puts it in the global stylesheet. To test, apply the patch and clear your browser cache if necessary. - Enable the patronimages system preference. - View a patron account which lacks a patron image. All views (details, fines, notices, etc) should show the "blank" patron image, including the modification log view. - View a patron account which has a patron image and check that it still displays correctly in all views. Signed-off-by: Josef Moravec Signed-off-by: Jonathan Druart --- koha-tmpl/intranet-tmpl/prog/css/staff-global.css | 18 ++++++++++++++++++ .../intranet-tmpl/prog/en/includes/circ-menu.inc | 8 ++++++-- koha-tmpl/intranet-tmpl/prog/img/patron-blank.min.svg | 1 + koha-tmpl/intranet-tmpl/prog/img/patron-blank.png | Bin 1559 -> 0 bytes 4 files changed, 25 insertions(+), 2 deletions(-) create mode 100644 koha-tmpl/intranet-tmpl/prog/img/patron-blank.min.svg delete mode 100644 koha-tmpl/intranet-tmpl/prog/img/patron-blank.png diff --git a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css index 35479c8..175fce8 100644 --- a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css +++ b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css @@ -535,6 +535,24 @@ div.patroninfo ul li { list-style-type : none; } +#patronbasics div { + background: transparent url("../img/patron-blank.min.svg") 10px 5px no-repeat; + border: 1px solid #CCCCCC; + height: 125px; + margin: .3em 0 .3em .3em; + padding: 0; + width: 105px; +} + +#patronimage { + border: 1px solid #CCCCCC; + max-width : 140px; + margin: .3em 0 .3em .3em; + padding: .2em; + width:auto !important; + width:130px; +} + div.patronviews { border-right : 1px solid #000; border-top : 1px solid #000; 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 083a88c..8d90fa9 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc @@ -8,9 +8,13 @@
    [% IF ( patronimages ) %] [% IF ( picture ) %] -
  • [% firstname | html %] [% surname | html %] ([% cardnumber | html %])
  • +
  • + [% firstname | html %] [% surname | html %] ([% cardnumber | html %]) +
  • [% ELSE %] -
  • [% firstname | html %] [% surname | html %] ([% cardnumber | html %])
  • +
  • +
    +
  • [% END %] [% END %] [% IF Koha.Preference( 'AddressFormat' ) %] diff --git a/koha-tmpl/intranet-tmpl/prog/img/patron-blank.min.svg b/koha-tmpl/intranet-tmpl/prog/img/patron-blank.min.svg new file mode 100644 index 0000000..795a74f --- /dev/null +++ b/koha-tmpl/intranet-tmpl/prog/img/patron-blank.min.svg @@ -0,0 +1 @@ + \ No newline at end of file diff --git a/koha-tmpl/intranet-tmpl/prog/img/patron-blank.png b/koha-tmpl/intranet-tmpl/prog/img/patron-blank.png deleted file mode 100644 index a6ea80f181c12ee351e5d072743ac87ff1d2f45f..0000000000000000000000000000000000000000 GIT binary patch literal 0 KcmV+b0RR6000031 literal 1559 zcmV+y2I%>TP)8F2MgRZ* z%*@Qh#KZsq0OjT7+1c68&(Hk){Os)P^78W4)z#nM-}UwNnVFgS`T2i;f6~&@GBPp@ z3=Hq@@8aU(W@cs?85!y6>87TpCMG6KOibL|+>DHjzrVk7a&kXEKeDp2Sy@>#2`;Vx z00lQmL_t(|+G1d$0{{ph1||lY115j~qKhQ}0Yn!|00M|EmH-40T`U0zAi7ur5I}UX z1R#LuVhKP1(Zv#g0HTW}00Bf7O8^3hE|vfU5M3+*2q3yx0uVqXS;8*N!U_Q_!t5ki z0uVqXSi-@@hzwXch_(bEfQYt)Lx2$-2oPxv6F>kFX$cP(BR0UrLzE=|0Ysc79Bho( z0UI%%VFCysqAX!y#0^AS~Baktf00M}hB_fQ30fA^?0tg^NmhdqW1NiV;0uVriEa4`~68v6a z0tg@imasDt2k^!U6F>kFu!Mt1OYkORCV&7UV2L!5mPq5Z1R#J2SRz2AC2V*t0SF)h zmM}6>AySwC0tmk)oJ3oKFV`~x1Q1;;0SF+vSOO40bg={=faqcgKmg&lgokKL@KyFq z00Bg;B#bwo00a;LOSp)%gA1=E00Bh65)mRT!P}f*0tg@imIxDRi7;ME00M|meZoeR z9Rm346Mz6BWC;sVmayQr1R#J2S;9_?R|vEum;eHZU_*i!uZR<9NB{&7;dU3{RLn}K z-31Upge~D=qp;bk|PMJYHi02p~L`aIi9x0N8L_!vqjOI4$82U?dR;;L4Xw00D%<5?!YcVE(0K%|Dgi<>g+0aW)CV&7!w}g+8a)937W&#Kx zbW6CYUwnQkVb&h!jg0#f11ll!PG25+R}!F%v)lkz@(4ER+GZg_mec z00M|4OW;6;PEf{0*Ee_00ay1bKy