Bug 16456: Add Font Awesome icons to some buttons in Tools module, section Patrons and circulation
Add Font Awesome Icons to section Patrons and circulation in Tools module. Also correct a error dialog in JQuery functions To test: -Apply patch -Goto Tools -> Patron list -> my_list_saved -> Add patrons -> Remove selected patrons, Clear all, Select all. -Add new patron list and add some patrons, notice about the trash icon. -Make some comments in some bib records and goto Tools->Comments you will presented with two tabs "Approved comments" and "Comments awaiting moderation". See the three new buttons: Approve, Delete, Unapprove. -Set syspref TagsModeration to Require. This will show all pending tags to review. -Make some tags in bib records and goto Tools -> Tags. -Notice about the new look. -In the new screen look the icons in buttons "Apply filter(s)", "Test", "Approved", "Reject". Notice about the new header bar above the DataTable with options: Select all, Clear all, Select all pending. -Play with filters; Check if terms exist or not in appoved/rejected lists -Play with Terms summary and see if DataTable is working as expected -Click in some term tag with multiple titles -A table with titles tagged with the term is presented -See the new button 'Remove' -Verify if you can remove tag from a selected title. -Verify that all tools work as expected NOTE: The Tag and Comments tools has been revised to fit with others interfaces in Koha. Bug ammended according with QA comment 5 Bug rebased because bug 16005 Test plan amended for clarity Clock icon for "Select all pending" removed (QA comment 23). Fix some forbidden patterns (tab char) in review.tt according by IRC comment by Marc Veron Followed test plan, looks and works as expected Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
This commit is contained in:
Hector Castro
parent
981302a473
commit
2a6a3de7c4
5 changed files with 69 additions and 49 deletions
|
|
@ -33,7 +33,7 @@ function patron_autocomplete(params) {
|
|||
}
|
||||
|
||||
function AddPatron( patron_name, value, container, input_name ) {
|
||||
div = "<div id='borrower_" + value + "'>" + patron_name + " ( <a href='#' class='removePatron'> " + MSG_REMOVE_PATRON + " </a> ) <input type='hidden' name='" + input_name + "' value='" + value + "' /></div>";
|
||||
div = "<div id='borrower_" + value + "'>" + patron_name + " ( <a href='#' class='removePatron'><i class='fa fa-trash' aria-hidden='true'></i> " + MSG_REMOVE_PATRON + " </a> ) <input type='hidden' name='" + input_name + "' value='" + value + "' /></div>";
|
||||
$(container).append( div );
|
||||
|
||||
$(container).parent().show( 800 );
|
||||
|
|
|
|||
|
|
@ -102,8 +102,8 @@ $(document).ready(function() {
|
|||
<form action="list.pl" id="remove_patrons" method="post">
|
||||
|
||||
<div id="searchheader">
|
||||
<span class="checkall"><a id="CheckAll" href="#">Select all</a></span> |
|
||||
<span class="clearall"><a id="CheckNone" href="#">Clear all</a></span> |
|
||||
<span class="checkall"><a id="CheckAll" href="#"><i class="fa fa-check"></i> Select all</a></span> |
|
||||
<span class="clearall"><a id="CheckNone" href="#"><i class="fa fa-remove"></i> Clear all</a></span> |
|
||||
<div class="btn-group">
|
||||
<button class="btn btn-mini list-remove" type="submit"><i class="fa fa-trash"></i> Remove selected</button>
|
||||
</div>
|
||||
|
|
@ -152,7 +152,7 @@ $(document).ready(function() {
|
|||
</table>
|
||||
|
||||
<input type="hidden" name="patron_list_id" value="[% list.patron_list_id %]" />
|
||||
<input type="submit" value="Remove selected patrons" />
|
||||
<button type="submit" class="btn btn-small"><i class="fa fa-trash" aria-hidden="true"></i> Remove selected patrons</button>
|
||||
</form>
|
||||
|
||||
</div>
|
||||
|
|
|
|||
|
|
@ -53,8 +53,8 @@
|
|||
[% review.review |html %]
|
||||
</td>
|
||||
<td>
|
||||
[% IF ( status ) %]<a href="/cgi-bin/koha/reviews/reviewswaiting.pl?op=unapprove&reviewid=[% review.reviewid %]">Unapprove</a>[% ELSE %]<a href="/cgi-bin/koha/reviews/reviewswaiting.pl?op=approve&reviewid=[% review.reviewid %]">Approve</a>[% END %] |
|
||||
<a href="/cgi-bin/koha/reviews/reviewswaiting.pl?op=delete&reviewid=[% review.reviewid %]">Delete</a>
|
||||
[% IF ( status ) %]<a href="/cgi-bin/koha/reviews/reviewswaiting.pl?op=unapprove&reviewid=[% review.reviewid %]" class="btn btn-mini"><i class="fa fa-remove"></i> Unapprove</a>[% ELSE %]<a href="/cgi-bin/koha/reviews/reviewswaiting.pl?op=approve&reviewid=[% review.reviewid %]" class="btn btn-mini"><i class="fa fa-check"></i> Approve</a>[% END %]
|
||||
<a href="/cgi-bin/koha/reviews/reviewswaiting.pl?op=delete&reviewid=[% review.reviewid %]" class="btn btn-mini"><i class="fa fa-trash" aria-hidden="true"></i> Delete</a>
|
||||
</td>
|
||||
</tr>
|
||||
[% END %]</tbody>
|
||||
|
|
|
|||
|
|
@ -71,7 +71,7 @@ tr.selected { background-color : #FFFFCC; } tr.selected td { background-color :
|
|||
</li>
|
||||
[% END %]</ul>[% ELSE %]This record has no items.[% END %]
|
||||
</td>
|
||||
<td><form method="post" action="/cgi-bin/koha/tags/list.pl"><input type="hidden" name="op" value="del" /><input type="hidden" name="tag" value="[% tag %]" /><input type="hidden" name="tag_id" value="[% title.tag_id %]" /><input type="submit" class="delete" value="Remove tag" /></form></td>
|
||||
<td><form method="post" action="/cgi-bin/koha/tags/list.pl"><input type="hidden" name="op" value="del" /><input type="hidden" name="tag" value="[% tag %]" /><input type="hidden" name="tag_id" value="[% title.tag_id %]" /><button type="submit" class="delete btn btn-small"><i class="fa fa-trash"></i> Remove</button></form></td>
|
||||
</tr>
|
||||
[% END %]
|
||||
</table>
|
||||
|
|
|
|||
|
|
@ -74,11 +74,11 @@ tr > td input,td input[type="submit"] { font-size: 85%; padding: 1px; }
|
|||
|
||||
var success_test_call = function() {
|
||||
$('#test_button').prop('disabled', false);
|
||||
$('#test_button').attr("value",_("Test"));
|
||||
$('#test_button').html("<i class='fa fa-check-square-o' aria-hidden='true'></i>" +_(" Test"));
|
||||
};
|
||||
$(document).ready(function() {
|
||||
$('.ajax_buttons' ).css({visibility:"visible"});
|
||||
$("p.check").html("<strong>"+_("Select:")+" <\/strong><a id=\"CheckAll\" href=\"/cgi-bin/koha/tags/review.pl\">"+_("All")+"<\/a> <a id=\"CheckPending\" href=\"/cgi-bin/koha/tags/review.pl\">"+_("Pending")+"<\/a> <a id=\"CheckNone\" href=\"/cgi-bin/koha/tags/review.pl\">"+_("None")+"<\/a>");
|
||||
$("p.check").html("<div id=\"searchheader\"><a id=\"CheckAll\" href=\"/cgi-bin/koha/tags/review.pl\"><i class=\"fa fa-check\"><\/i>"+_(" Select all")+"<\/a> | <a id=\"CheckNone\" href=\"/cgi-bin/koha/tags/review.pl\"><i class=\"fa fa-remove\"><\/i>"+_(" Clear all")+"<\/a> | <a id=\"CheckPending\" href=\"/cgi-bin/koha/tags/review.pl\"> "+_("Select all pending")+"<\/a><\/div>");
|
||||
$("#CheckAll").click(function(){
|
||||
$(".checkboxed").checkCheckboxes();
|
||||
return false;
|
||||
|
|
@ -96,28 +96,48 @@ tr > td input,td input[type="submit"] { font-size: 85%; padding: 1px; }
|
|||
// window.alert(_("Click detected on ") + event.target + ": " + $(event.target).html);
|
||||
if ($(event.target).is('.ok')) {
|
||||
$.ajax({
|
||||
"data": {ok: $(event.target).attr("title")},
|
||||
"success": count_approve // success_approve
|
||||
data: {ok: $(event.target).attr("title")},
|
||||
success: count_approve // success_approve
|
||||
});
|
||||
$(event.target).next(".rej").prop('disabled', false).attr("value",_("Reject")).css("color","#000");
|
||||
$(event.target).attr( "value",_("Approved")).prop('disabled', true).css("color","#666");
|
||||
$(event.target).next(".rej").prop('disabled', false).css("color","#000");
|
||||
$(event.target).next(".rej").html("<i class='fa fa-remove'></i>" + _(" Reject"));
|
||||
$(event.target).prop('disabled', true).css("color","#666");
|
||||
$(event.target).html("<i class='fa fa-check'></i>" + _(" Approved"));
|
||||
var getelement = $(event.target).attr("title");
|
||||
var gettitle = "span[data-title|=" + getelement + "]";
|
||||
$(gettitle).text(_("Approved"));
|
||||
if ($(gettitle).attr("class") == "pending"){
|
||||
$(gettitle).toggleClass("pending approved");
|
||||
} else {
|
||||
$(gettitle).toggleClass("rejected approved");
|
||||
}
|
||||
return false; // cancel submit
|
||||
}
|
||||
if ($(event.target).is('.rej')) {
|
||||
$.ajax({
|
||||
"data": {rej: $(event.target).attr("title")},
|
||||
"success": count_reject // success_reject
|
||||
});
|
||||
$(event.target).prev(".ok").prop('disabled', false).attr("value",_("Approve")).css("color","#000");
|
||||
$(event.target).attr( "value",_("Rejected")).prop('disabled', true).css("color","#666");
|
||||
data: {rej: $(event.target).attr("title")},
|
||||
success: count_reject // success_reject
|
||||
});
|
||||
$(event.target).prev(".ok").prop('disabled', false).css("color","#000");
|
||||
$(event.target).prev(".ok").html("<i class='fa fa-check'></i>" + _(" Approve"));
|
||||
$(event.target).prop('disabled', true).css("color","#666");
|
||||
$(event.target).html("<i class='fa fa-remove'></i>" + _(" Rejected"));
|
||||
var getelement = $(event.target).attr("title");
|
||||
var gettitle = "span[data-title|=" + getelement + "]";
|
||||
$(gettitle).text(_("Rejected"));
|
||||
if ($(gettitle).attr("class") == "pending"){
|
||||
$(gettitle).toggleClass("pending rejected");
|
||||
} else {
|
||||
$(gettitle).toggleClass("approved rejected");
|
||||
}
|
||||
return false; // cancel submit
|
||||
}
|
||||
if ($(event.target).is('#test_button')) {
|
||||
$(event.target).attr( "value",_("Testing...")).prop('disabled', true);
|
||||
$.ajax({
|
||||
"data": {test: $('#test').attr("value")},
|
||||
"success": success_test_call // success_reject
|
||||
});
|
||||
if ($(event.target).is('#test_button')) {
|
||||
$(event.target).text(_("Testing...")).prop('disabled', true);
|
||||
$.ajax({
|
||||
data: {test: $('#test').attr("value")},
|
||||
success: success_test_call // success_reject
|
||||
});
|
||||
return false; // cancel submit
|
||||
}
|
||||
});
|
||||
|
|
@ -239,11 +259,11 @@ tr > td input,td input[type="submit"] { font-size: 85%; padding: 1px; }
|
|||
</td>
|
||||
<td>
|
||||
[% IF ( tagloo.approved == -1 ) %]
|
||||
<span class="rejected">Rejected</span>
|
||||
<span class="rejected" data-title="[% tagloo.term %]">Rejected</span>
|
||||
[% ELSIF ( tagloo.approved == 1 ) %]
|
||||
<span class="approved">Approved</span>
|
||||
<span class="approved" data-title="[% tagloo.term %]">Approved</span>
|
||||
[% ELSE %]
|
||||
<span class="pending">Pending</span>
|
||||
<span class="pending" data-title="[% tagloo.term %]">Pending</span>
|
||||
[% END %]
|
||||
</td>
|
||||
<td><a href="/cgi-bin/koha/tags/list.pl?tag=[% tagloo.term|uri %]">[% tagloo.term %]</a>
|
||||
|
|
@ -253,11 +273,11 @@ tr > td input,td input[type="submit"] { font-size: 85%; padding: 1px; }
|
|||
[% IF ( tagloo.approved ) %]
|
||||
<td class="actions"><span class="ajax_buttons" style="visibility:hidden">
|
||||
[% IF ( tagloo.approved == -1 ) %]
|
||||
<input class="ok" type="submit" title="[% tagloo.term %]" value="Approve" name="approve" />
|
||||
<input class="rej" disabled="disabled" type="submit" title="[% tagloo.term %]" value="Rejected" name="reject" />
|
||||
<button class="ok btn btn-mini" type="submit" title="[% tagloo.term %]" value="Approve" name="approve"><i class="fa fa-check"></i> Approve</button>
|
||||
<button class="rej btn btn-mini" disabled="disabled" type="submit" title="[% tagloo.term %]" value="Rejected" name="reject"><i class="fa fa-remove"></i> Rejected</button>
|
||||
[% ELSE %]
|
||||
<input class="ok" disabled="disabled" type="submit" title="[% tagloo.term %]" value="Approved" name="approve" />
|
||||
<input class="rej" type="submit" title="[% tagloo.term %]" value="Reject" name="reject" />
|
||||
<button class="ok btn btn-mini" disabled="disabled" type="submit" title="[% tagloo.term %]" value="Approved" name="approve"><i class="fa fa-check"></i> Approved</button>
|
||||
<button class="rej btn btn-mini" type="submit" title="[% tagloo.term %]" value="Reject" name="reject"><i class="fa fa-remove"></i> Reject</button>
|
||||
[% END %]
|
||||
</span>
|
||||
</td>
|
||||
|
|
@ -265,8 +285,8 @@ tr > td input,td input[type="submit"] { font-size: 85%; padding: 1px; }
|
|||
</td>
|
||||
[% ELSE %]
|
||||
<td class="actions"><span class="ajax_buttons" style="visibility:hidden">
|
||||
<input class="ok" type="submit" title="[% tagloo.term %]" value="Approve" name="approve" />
|
||||
<input class="rej" type="submit" title="[% tagloo.term %]" value="Reject" name="reject" />
|
||||
<button class="ok btn btn-mini" type="submit" title="[% tagloo.term %]" value="Approve" name="approve"><i class="fa fa-check"></i> Approve</button>
|
||||
<button class="rej btn btn-mini" type="submit" title="[% tagloo.term %]" value="Reject" name="reject"><i class="fa fa-remove"></i> Reject</button>
|
||||
</span>
|
||||
</td>
|
||||
[% UNLESS ( filter_approved_pending ) %]<td> </td>[% END %]
|
||||
|
|
@ -280,8 +300,8 @@ tr > td input,td input[type="submit"] { font-size: 85%; padding: 1px; }
|
|||
</table>
|
||||
[% END %]
|
||||
[% IF ( tagloop ) %]<fieldset class="action">
|
||||
<input type="submit" value="Approve" id="approve_button" name="op-approve" />
|
||||
<input type="submit" value="Reject" id="reject_button" name="op-reject" />
|
||||
<button type="submit" value="Approve" class="btn btn-small" id="approve_button" name="op-approve"><i class="fa fa-check"></i> Approve</button>
|
||||
<button type="submit" value="Reject" class="btn btn-small" id="reject_button" name="op-reject"><i class="fa fa-remove"></i> Reject</button>
|
||||
</fieldset>[% END %]
|
||||
</form>
|
||||
</div>
|
||||
|
|
@ -289,18 +309,18 @@ tr > td input,td input[type="submit"] { font-size: 85%; padding: 1px; }
|
|||
<fieldset class="brief">
|
||||
<h4>Terms summary</h4>
|
||||
<ul>
|
||||
<li><a href="/cgi-bin/koha/tags/review.pl?approved=1">Approved</a>:
|
||||
<span id="terms_summary_approved_count">[% approved_count %]</span>
|
||||
</li>
|
||||
<li><a href="/cgi-bin/koha/tags/review.pl?approved=-1">Rejected</a>:
|
||||
<span id="terms_summary_rejected_count">[% rejected_count %]</span>
|
||||
</li>
|
||||
<li><a href="/cgi-bin/koha/tags/review.pl?approved=0">Pending</a>:
|
||||
<span id="terms_summary_unapproved_count">[% unapproved_count %]</span>
|
||||
</li>
|
||||
<li><a href="/cgi-bin/koha/tags/review.pl?approved=all">Total</a>:
|
||||
<span id="terms_summary_approved_total">[% approved_total %]</span>
|
||||
</li>
|
||||
<li><a href="/cgi-bin/koha/tags/review.pl?approved=1">Approved</a>:
|
||||
<span id="terms_summary_approved_count">[% approved_count %]</span>
|
||||
</li>
|
||||
<li><a href="/cgi-bin/koha/tags/review.pl?approved=-1">Rejected</a>:
|
||||
<span id="terms_summary_rejected_count">[% rejected_count %]</span>
|
||||
</li>
|
||||
<li><a href="/cgi-bin/koha/tags/review.pl?approved=0">Pending</a>:
|
||||
<span id="terms_summary_unapproved_count">[% unapproved_count %]</span>
|
||||
</li>
|
||||
<li><a href="/cgi-bin/koha/tags/review.pl?approved=all">Total</a>:
|
||||
<span id="terms_summary_approved_total">[% approved_total %]</span>
|
||||
</li>
|
||||
</ul>
|
||||
<span id="terms_summary_status"> </span>
|
||||
</fieldset>
|
||||
|
|
@ -309,7 +329,7 @@ tr > td input,td input[type="submit"] { font-size: 85%; padding: 1px; }
|
|||
<div class="description">Enter a word or phrase to check against approved/rejected lists: </div>
|
||||
<form method="post" action="/cgi-bin/koha/tags/review.pl">
|
||||
<input type="text" size="14" name="test" id="test" />
|
||||
<fieldset class="action"><input type="submit" value="Test" id="test_button" name="op-test" /></fieldset>
|
||||
<fieldset class="action"><button type="submit" value="Test" class="btn btn-small" id="test_button" name="op-test"><i class="fa fa-check-square-o" aria-hidden="true"></i> Test</button></fieldset>
|
||||
<div id="verdict">
|
||||
[% IF ( test_term ) %]
|
||||
[% IF ( verdict_ok ) %]
|
||||
|
|
@ -360,7 +380,7 @@ tr > td input,td input[type="submit"] { font-size: 85%; padding: 1px; }
|
|||
<div class="hint">[% INCLUDE 'date-format.inc' %]</div>
|
||||
</li>
|
||||
</ol>
|
||||
<fieldset class="action"><input type="submit" value="Apply filter(s)" /></fieldset>
|
||||
<fieldset class="action"><button type="submit" class="btn btn-small"><i class="fa fa-filter" aria-hidden="true"></i> Apply filter(s)</button></fieldset>
|
||||
</fieldset>
|
||||
</form>
|
||||
</div>
|
||||
|
|
|
|||
Loading…
Reference in a new issue