https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20312

--- Comment #9 from Owen Leonard <[email protected]> ---
Some suggestions:

 - Use <i class="fa fa-remove"></i> instead of an image for X.
 - Put the CSS in staff-global.scss
 - Put the JavaScript in staff-global.js
   - Add the JS to the existing $(document).ready() so you don't need
'addEventListener('DOMContentLoaded')';
   - Be consistent about whether you use single or double quotes

(In reply to Katrin Fischer from comment #8)

> I wonder if libraries would like to have this optional for privacy reasons.
> It might be safer to add a pref.

A lot of this already rests in the hands of the librarian and their browser. If
I have my browser configured to restore my last session, my back button will
take me back to the last patron without a handy link.

The way this patch does it, the cookie should disappear when the browser is
closed. I would think that is adequate.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to