http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11812
Owen Leonard <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25560|0 |1 is obsolete| | --- Comment #4 from Owen Leonard <[email protected]> --- Created attachment 32302 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=32302&action=edit [SIGNED-OFF] Bug 11812 - Add missing "required" indicator to fields which are required [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 <[email protected]> QA Edits: Re-worded the "required" not on the batch patron edit form, added a missing word to the help text on that page. On the csv-profiles page I removed an unnecessary "javascript:" protocol from the markup. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] 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/
