---- Reported by nengard@gmail.com 2009-12-16 00:41:48 ---- It would be nice to move the 'test prediction pattern' button up under the prediction box (shouldn't have to scroll to bottom of page) --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:21 UTC --- This bug was previously known as _bug_ 3934 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3934 Actual time not defined. Setting to 0.0 Setting qa contact to the default for this product. This bug either had no qa contact or an invalid one.
Created attachment 20484 [details] [review] Bug 3934 - move test prediction pattern button On the subscription add/edit screen the buttons related to numbering patterns are grouped with the "save" button, which is potentially confusing. This patch moves the buttons into the "serials planning" region so they are visually grouped with related functionality. Unrelated changes: Removed a few redundant "javascript:" pseudo-protocols from event handlers, and added a "Cancel" link. To test, edit an existing subscription and confirm that the prediction pattern buttons are grouped inside the "serials planning" region. Confirm that each button triggers the correct event. Confirm that clicking the "cancel" link returns you to the subscription detail view. Also test creating a new subscription. Again buttons should work correctly. The "Cancel" link should return you to the Serials module home page.
This template has been completely rewritten by bug 7688 (Passed QA). Maybe it would be wise to wait until it is pushed to master.
Patch applied cleanly, go forth and signoff
Created attachment 21882 [details] [review] move test prediction pattern button I confirm that the patch works well
Note for QAers: Given that bug 7688 is ready for the v.3.14 release, this patch won't be valid.
After 7688 has been pushed, I think this one will become irrelevant. Switching to "in discussion", but imo, it could be RESO WONTFIX
It could still be a nice enhancement for 3.12 and earlier.
Created attachment 22429 [details] [review] Bug 3934 - move test prediction pattern button On the subscription add/edit screen the buttons related to numbering patterns are grouped with the "save" button, which is potentially confusing. This patch moves the buttons into the "serials planning" region so they are visually grouped with related functionality. Unrelated changes: Removed a few redundant "javascript:" pseudo-protocols from event handlers, and added a "Cancel" link. To test, edit an existing subscription and confirm that the prediction pattern buttons are grouped inside the "serials planning" region. Confirm that each button triggers the correct event. Confirm that clicking the "cancel" link returns you to the subscription detail view. Also test creating a new subscription. Again buttons should work correctly. The "Cancel" link should return you to the Serials module home page. Signed-off-by: Paola Rossi <paola.rossi@cineca.it> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
This patch has been pushed to 3.12.x, will be in 3.12.8. Thanks Owen!