Summary: | Improve clarity of batch modification operations | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Tools | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Jonathan Druart <jonathan.druart> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | jonathan.druart, nengard, paul.poulain |
Version: | 3.10 | ||
Hardware: | All | ||
OS: | All | ||
URL: | /cgi-bin/koha/tools/batchMod.pl | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 7784 - Improve clarity of batch modification operations
[SIGNED-OFF] Bug 7784 - Improve clarity of batch modification operations |
Description
Owen Leonard
2012-03-21 16:08:12 UTC
Created attachment 11709 [details] [review] Bug 7784 - Improve clarity of batch modification operations - Improve clarity of hint about checkboxes - Inicate that leaving fields blank will make no change - Add a hint when checking checkboxes stating that the field will be deleted. - Block deletion of required subfields - Add "required" style to required subfields (matching additem.pl) Created attachment 11712 [details] [review] [SIGNED-OFF] Bug 7784 - Improve clarity of batch modification operations - Improve clarity of hint about checkboxes - Inicate that leaving fields blank will make no change - Add a hint when checking checkboxes stating that the field will be deleted. - Block deletion of required subfields - Add "required" style to required subfields (matching additem.pl) Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Works as described QA Comment: Just one question: what is the utility of the last line of your patch: + <span class="hint" id="hint[% ite.tag %][% ite.subfield %][% ite.random %]"></span> Else nothing to say, works as expected, no regression. Marked as Passed QA Patch pushed to master (In reply to comment #3) > Just one question: what is the utility of the last line of your patch: > + <span class="hint" id="hint[% ite.tag %][% ite.subfield > %][% ite.random %]"></span> The hint is shown if you check the box to delete a subfield: "This subfield will be deleted." The ID is attached so that if you uncheck the box the hint can be removed again. |