Bug 10418 - Correct template markup problems on course reserves add item page
authorOwen Leonard <oleonard@myacpl.org>
Wed, 5 Jun 2013 13:56:23 +0000 (09:56 -0400)
committerGalen Charlton <gmc@esilibrary.com>
Fri, 14 Jun 2013 18:21:45 +0000 (11:21 -0700)
commit5fbf07df7ccd404570281256ce8f9afb5a09834c
tree18a8fc27fb105f5e9ab53a605489be05e76ce3b4
parent02d4719437a84ca99d6265ad5bc27a6a8c3d88ad
Bug 10418 - Correct template markup problems on course reserves add item page

This patch corrects several problems with the course reserves add item
pages:

- Non-standard grid markup.
- Extra </div>.
- Incorrect styling of errors and messages.
- Redundant JS for adding focus to a field (use "focus" class).
- Invalid use of <label> with no corresponding input (use <span
  class="label"> instead).
- Incorrect capitalization.

To test, apply the patch and examing both steps of the process of adding
an item to an existing course: The initial "scan barcode" form, and the
form for editing details of a scanned item.

Some cases to check:

- Scan an invalid barcode.
- Scan a barcode which has already been added to a course.
- Scan a barcode wihch has already been added to the course you're
  working with.

Forms should look correct and work correctly.

This patch contains whitespace changes, so please ignore whitespace when
examining changes.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
The add item screen continues to work as before.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Passes koha-qa.pl. Works as advertised.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/add_items-step1.tt
koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/add_items-step2.tt