[Koha-bugs] [Bug 11812] Add missing "required" indicator to fields which are required
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Sun Feb 23 21:40:49 CET 2014
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11812
--- Comment #2 from Marc Véron <veron at veron.ch> ---
Created attachment 25560
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25560&action=edit
[SIgned-off] Bug 11812 - Add missing "required" indicator to fields which are
required
Form fields in the staff client which are required should be configured
to be so by doing several things:
- Add a class "required" to the field's <label>
- Add a class "required" to the form field
- Add 'required="required"' to the form field
- Apppend a <span class="required">Required</span> after the form field.
Several places in the templates are missing the <span>. This patch adds
them.
To test, apply the patch and view the following pages to confirm that
the "Required" text appears:
- Acquisitions -> Add an order to a basket from a new (empty) record.
Title, quantity, and fund should indicate they are required.
- Administration -> Authority types -> New authority type. The
authority type and description fields should indicate they are
required.
- Administration -> Authority types -> MARC structure -> New tag. The
tag field should indicate it is required.
- Patron types and categories -> New category. Category code,
description, and category type should indicate that they are required.
FIXME: Enrollment period is required but the user must choose one. I'm
not sure how to handle that clearly.
- Tools -> CSV profiles. Profile name, profile type, and profile MARC
fields should indicate they are required on both the new and edit
forms.
- Administration -> Manage MARC modification templates. Under "Create a
new template" the name field should indicate that it is required.
- Tools -> Batch patron modification -> Submit a batch for editing. Any
fields which are required according to your BorrowerMandatoryField
system preference should indicate that they are required.
Patch behaves as expected.
Signed-off-by: Marc Véron <veron at veron.ch>
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list