Bug 21942: Clean up cataloging merge template

This patch makes a few minor markup corrections to the cataloging merge
template:

 - Remove invalid "type" attributes from <style> and <script>
 - Add missing closing </span>
 - Remove obsolete YUI grid markup

To test, apply the patch and test the process of merging records in
Cataloging. Everything should look correct and work correctly.
Validating the HTML source should return no errors stemming from the
template.

Signed-off-by: Michal Denar <black23@gmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
This commit is contained in:
Owen Leonard 2018-12-04 15:32:17 +00:00 committed by Nick Clemens
parent 87026c6cad
commit c54c12f53d
1 changed files with 2 additions and 4 deletions

View File

@ -155,7 +155,6 @@ div#result { margin-top: 1em; }
[% ELSE %]
<form id="mergeform" action="/cgi-bin/koha/cataloguing/merge.pl" method="post" onsubmit="return mergeformsubmit()">
<div class="yui-g">
<div class="col-sm-6">
[% PROCESS mergesource sourcerecords=records %]
</div>
@ -173,9 +172,8 @@ div#result { margin-top: 1em; }
<input type="submit" name="merge" value="Merge" />
<label for="report_fields">Fields to display in report:</label>
<input type="text" name="report_fields" id="report_fields" value="[% MergeReportFields | html %]" />
<span class="hint">(Example: "001,245ab,600")
<span class="hint">(Example: "001,245ab,600")</span>
</fieldset>
</div>
</form>
[% END %]
[% END %]
@ -187,7 +185,7 @@ div#result { margin-top: 1em; }
[% MACRO jsinclude BLOCK %]
[% Asset.js("js/merge-record.js") | $raw %]
[% INCLUDE 'merge-record-strings.inc' %]
<script type="text/javascript">
<script>
[% UNLESS (result) %]
[% IF (choosereference) %]
function changeFramework(fw) {