Bug 15949 - Making certain alerts stand out
Summary: Making certain alerts stand out
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords: Academy
Depends on:
Blocks:
 
Reported: 2016-03-01 17:14 UTC by Christopher Brannon
Modified: 2020-01-09 00:04 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Christopher Brannon 2016-03-01 17:14:36 UTC
In the attempt to make certain messages stand out from others (in this case, when an item is checked in that is not in the system or is miss-scanned), I added the following jquery:

//Enhance No item with barcode message
$("p.problem:contains('No item with barcode:')").parent("div").removeClass("alert").css("background-color","red").css("border","1px solid #e0c726").css("background","rgba(0, 0, 0, 0) -moz-linear-gradient(center top , #FAECEC 0%, #FAECEC 9%, #FC1B1B 89%, #FA2A2A 100%) repeat scroll 0 0");
//End Enhance

Would love to see visual enhancements of the alerts become a preference in admin.

Christopher
Comment 1 Owen Leonard 2016-03-01 17:25:30 UTC
I think perhaps a better solution is to make sure that messages like this get unique ID atributes so that you can define custom CSS for particular messages which are of higher priority to your library. For example:

[% IF ( errmsgloo.badbarcode ) %]
    <p id="checkin_no_item_with_barcode" class="problem">No item with barcode: [% errmsgloo.msg %]</p>
[% END %]

That would also allow you to define custom audio alerts for those messages.
Comment 2 Katrin Fischer 2020-01-08 23:50:58 UTC
(In reply to Owen Leonard from comment #1)
> I think perhaps a better solution is to make sure that messages like this
> get unique ID atributes so that you can define custom CSS for particular
> messages which are of higher priority to your library. For example:
> 
> [% IF ( errmsgloo.badbarcode ) %]
>     <p id="checkin_no_item_with_barcode" class="problem">No item with
> barcode: [% errmsgloo.msg %]</p>
> [% END %]
> 
> That would also allow you to define custom audio alerts for those messages.

+1
Comment 3 Christopher Brannon 2020-01-09 00:04:06 UTC
I think, in the interest of keeping things simple, and not having programmers hate this, it would be better to have either a brief code, or a numeric problem ID or subclass so people are not having to use #checkin_no_item_with_barcode.  As someone that uses jQuery a lot, it would annoy me more than keeping a unique code or number straight.  But, that is my personal opinion and not the opinion of programmers.

Either way, it would work.

HOWEVER, it would be even BETTER, if there were a page that could be dedicated to customizations, and would isolate key alerts and such that people could easily set.  Koha is HIGHLY customizable, but not everyone wants the nitty gritty of CSS.  Looking things up, possibly mistyping something, trying to figure out what they did wrong.  Not everyone wants to keep up on CSS.

Keep the ability to customize CSS, but add some basic choices for those who don't CSS.