[Bug 15950] New: Use Font Awesome icons for acquisitions basket close confirmation
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15950 Bug ID: 15950 Summary: Use Font Awesome icons for acquisitions basket close confirmation Change sponsored?: --- Product: Koha Version: master Hardware: All URL: /cgi-bin/koha/acqui/basket.pl?op=close OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: oleonard@myacpl.org QA Contact: testopia@bugs.koha-community.org Depends on: 15785 In Acquisitions, when you close a basket you are asked to confirm the operation. The dialog which asks you to confirm should use Font Awesome icons. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15785 [Bug 15785] Use Font Awesome icons in confirmation dialogs -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15950 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15950 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 48517 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48517&action=edit Bug 15950 - Use Font Awesome icons for acquisitions basket close confirmation When closing a basket in Acquisitions the confirmation dialog should be styled using Font Awesome icons. This patch adds them. The cancel button has been moved into its own form so that we can get rid of several problematic JavaScript issues: The use of the "javascript:" pseudo-protocol, the use of an "onclick" attribute, and the use of a JS redirect where a GET action will do. To test, apply the patch and locate an open basket in Acquisitions which can be closed. - Click the 'Close this basket' button and confirm that the icons in the dialog look correct. - Test the cancel operation to make sure you are redirected back to the page showing the basket details. - Test the confirm operation and verify that the basket is closed. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15950 Héctor Eduardo Castro Avalos <hector.hecaxmmx@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #48517|0 |1 is obsolete| | --- Comment #2 from Héctor Eduardo Castro Avalos <hector.hecaxmmx@gmail.com> --- Created attachment 48518 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48518&action=edit [SIGNED-OFF]Bug 15950: Use Font Awesome icons for acquisitions basket close confirmation When closing a basket in Acquisitions the confirmation dialog should be styled using Font Awesome icons. This patch adds them. The cancel button has been moved into its own form so that we can get rid of several problematic JavaScript issues: The use of the "javascript:" pseudo-protocol, the use of an "onclick" attribute, and the use of a JS redirect where a GET action will do. To test, apply the patch and locate an open basket in Acquisitions which can be closed. - Click the 'Close this basket' button and confirm that the icons in the dialog look correct. - Test the cancel operation to make sure you are redirected back to the page showing the basket details. - Test the confirm operation and verify that the basket is closed. Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as advertised -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15950 Héctor Eduardo Castro Avalos <hector.hecaxmmx@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |hector.hecaxmmx@gmail.com Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15950 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15950 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #48518|0 |1 is obsolete| | --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 48582 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48582&action=edit [PASSED QA] Bug 15950: Use Font Awesome icons for acquisitions basket close confirmation When closing a basket in Acquisitions the confirmation dialog should be styled using Font Awesome icons. This patch adds them. The cancel button has been moved into its own form so that we can get rid of several problematic JavaScript issues: The use of the "javascript:" pseudo-protocol, the use of an "onclick" attribute, and the use of a JS redirect where a GET action will do. To test, apply the patch and locate an open basket in Acquisitions which can be closed. - Click the 'Close this basket' button and confirm that the icons in the dialog look correct. - Test the cancel operation to make sure you are redirected back to the page showing the basket details. - Test the confirm operation and verify that the basket is closed. Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as advertised Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Please note that the confirmation message display depends on BasketConfirmations. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15950 Brendan Gallagher <brendan@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |brendan@bywatersolutions.co | |m Status|Passed QA |Pushed to Master --- Comment #4 from Brendan Gallagher <brendan@bywatersolutions.com> --- Pushed to Master - Should be in the May 2016 release. Thanks -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org