http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12944

--- Comment #18 from Katrin Fischer <katrin.fisc...@bsz-bw.de> ---
Comment on attachment 33197
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=33197
Bug 12944: Refactor the patron autocomplete

Review of attachment 33197:
 --> 
(http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=12944&attachment=33197)
-----------------------------------------------------------------

Hi Jonathan,

overall this looks good :)

::: koha-tmpl/intranet-tmpl/js/autocomplete/patrons.js
@@ +32,5 @@
> +    });
> +}
> +
> +function AddPatron( patron_name, value, container, input_name ) {
> +    div = "<div id='borrower_" + value + "'>" + patron_name + " ( <a 
> href='#' class='removePatron'> " + _("Remove") + " </a> ) <input 
> type='hidden' name='" + input_name + "' value='" + value + "' /></div>";

But by moving the Javascript into a separate js file, the "Remove" will no
longer be translatable. Could you please fix?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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