Wishlist: Selfcheck Hide UI Elements until patron auth

Bug #1466173 reported by Josh Stompro
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

Currently the EG web based self check shows several UI elements that are not usable even when no patron is authenticated. I would like all those UI elements to be hidden when the station is sitting idle, and then shown after a successful patron login, and then hidden again after patron logoff.

Additionally, I would like an easy way to customize the blank space that is left. This space could be used in numerous ways.

 * Welcome message from the Library.
 * Announcements from the Library.
 * Instructional text, images, gifs, videos that show how to use the self check.
 * Advertisement for upcoming events or services.
 * Book Carousels to advertise materials.

I attached an image showing the selfcheck UI in it's normal idle state and an example of the unusable UI hidden.

Thanks
Josh

Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :
Revision history for this message
Terran McCanna (tmccanna) wrote :

I did this for our local implementation, I will put together a fix to submit.

Changed in evergreen:
assignee: nobody → Terran McCanna (tmccanna)
Revision history for this message
Terran McCanna (tmccanna) wrote :

I have submitted a small fix for the "hiding elements" portion of this ticket:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=6967c0968c2f98e979101be595f6aa554cfb6112

For testing, simply apply and then log in / log out as a patron to be sure it is hiding the bottom content unless authenticated.

This fix does not include a mechanism for adding in easily updatable content, but I think for a local customization you could probably add a new DIV on main.tt2 that contains the content (maybe through an iframe so it could pull in a page from your library's web site?) and then add another line to the selfcheck.js to hide that div at the same time that the oils-selfck-bottom-div is being hidden:

openils.Util.hide('my-new-div');

Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
tags: added: pullrequest
Revision history for this message
Galen Charlton (gmc) wrote :

I've pushed the patch to master. Thanks, Terran!

Josh, if you want to pursue adding a mechanism to push content to the self-check page in a way other than via a template customization, please open a new bug.

Changed in evergreen:
milestone: none → 2.10-beta
status: New → Confirmed
importance: Undecided → Wishlist
tags: added: rm-to-write-notes
Changed in evergreen:
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.