Bugzilla – Attachment 112694 Details for
Bug 26838
Improve styling of checkin message
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 26838: Improve styling of checkin message
Bug-26838-Improve-styling-of-checkin-message.patch (text/plain), 5.82 KB, created by
Owen Leonard
on 2020-10-30 13:29:51 UTC
(
hide
)
Description:
Bug 26838: Improve styling of checkin message
Filename:
MIME Type:
Creator:
Owen Leonard
Created:
2020-10-30 13:29:51 UTC
Size:
5.82 KB
patch
obsolete
>From f3d88d795b0f02dfe9190d217ae0aa16ed188c65 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Fri, 30 Oct 2020 12:33:54 +0000 >Subject: [PATCH] Bug 26838: Improve styling of checkin message > >This patch makes some changes to the style and use of the "problem" >class in the staff interface: > >- Remove the background color >- Remove the extra line height > >Some changes to the template: > > - Remove the problem class from the checkin message. The checkin > message configuration determines whether the dialog is an alert or a > message-style dialog. > - Add Font Awesome icons to the "problem" messages in patron details > and serials collection. This is to help emphasize the message for > users who might have difficulty seeing the color difference. > >To test, apply the patch and rebuild the staff interface CSS >(https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client). > > - Go to Administration -> Item types and configure a checkin message > for an item type: Add a message and select "Message" as the checkin > message type. > - Check in an item with that item type and confirm that you see the > checkin message. It should be displayed in a message-style dialog > without any additional styling. > - Change the item type configuration to use "Alert" as the checkin > message type. Confirm that the correct style is shown in this case as > well. > > - Locate a patron with a NULL value in borrowers.password. View the > detail page for that patron. > - Under the "Library use" heading, the "Password" line should contain a > link with the text "Undefined," preceded by a red-colored Font > Awesome icon. > > - In Serials, locate a subscription which is expired. > - View the details for the subscription, and then open "serial > collection" from the sidebar menu. > - On the serial collection page there should be a cell in the > subscription summary table showing the message "Subscription expired" > preceded by a red-colored Font Awesome icon. >--- > koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss | 10 ---------- > koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt | 2 +- > koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt | 2 +- > .../prog/en/modules/serials/serials-collection.tt | 2 +- > 4 files changed, 3 insertions(+), 13 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 a698ec9bf9..82f9048aa8 100644 >--- a/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss >+++ b/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss >@@ -596,10 +596,8 @@ ol { > } > > .problem { >- background-color: #FFFFCC; > color: #990000; > font-weight: bold; >- line-height: 1.7em; > } > > fieldset { >@@ -2285,19 +2283,11 @@ li { > &.alert { > background: #FFFADE none; > border-color: #E0C726; >- >- .problem { >- background-color: transparent; >- } > } > > &.message { > background: #E8EDF6 none; > border-color: #A4BEDD; >- >- .problem { >- background-color: transparent; >- } > } > } > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >index b52fc5becf..b3a2bc857f 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >@@ -302,7 +302,7 @@ > [% ELSE %] > <div class="dialog message"> > [% END %] >- <p class="problem ret_checkinmsg">[% checkinmsg | html_line_break %]</p> >+ <p class="ret_checkinmsg">[% checkinmsg | html_line_break %]</p> > </div> > [% END # /IF checkinmsg %] > [% END # /BLOCK all_checkin_messages %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt >index 4bf7ba95f7..70a203086a 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt >@@ -544,7 +544,7 @@ > [% IF ( patron.password ) %] > ******* > [% ELSE %] >- <span class="problem"><a href="/cgi-bin/koha/members/member-password.pl?member=[% patron.borrowernumber | uri %]">Undefined</a></span> >+ <i class="fa fa-exclamation-circle problem" aria-hidden="true"></i> <a href="/cgi-bin/koha/members/member-password.pl?member=[% patron.borrowernumber | uri %]">Undefined</a> > [% END %] > </li> > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt >index 1aa688293a..6871b02007 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt >@@ -87,7 +87,7 @@ > <td> [% subscription.notes | html | html_line_break %] > [% UNLESS subscription.closed %] > [% IF ( subscription.subscriptionexpired ) %] >- <br /><span class="problem"> Subscription expired</span> >+ <p class="problem"><i class="fa fa-exclamation-circle problem" aria-hidden="true"></i> Subscription expired</p> > [% END %] > [% ELSE %] > <br /> Subscription closed >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 26838
:
112596
|
112694
|
112727
|
115046
|
121809
|
122170
|
122223