Bug 15785

Summary: Use Font Awesome icons in confirmation dialogs
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: brendan, hector.hecaxmmx, jonathan.druart, veron
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Medium patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 15784, 15790, 15804, 15821, 15826, 15858, 15884, 15950, 15951, 15959, 15960, 15961, 15963, 15978, 15979, 15980, 15983    
Attachments: Screenshot showing the kind of dialog to be modified
Bug 15785 - Use Font Awesome icons in confirmation dialogs
[SIGNED-OFF]Bug 15785 - Use Font Awesome icons in confirmation dialogs
Bug 15785 - Use Font Awesome icons in confirmation dialogs
Bug 15785 - Use Font Awesome icons in confirmation dialogs
Bug 15784: Library deletion warning is incorrectly styled

Description Owen Leonard 2016-02-10 18:22:16 UTC
Created attachment 47863 [details]
Screenshot showing the kind of dialog to be modified

Some (non-JavaScript) dialogs in Koha show a pair of Yes/No buttons styled with "approve"/"deny" classes which use image files for the icon. These could instead be styled using Font Awesome icons.
Comment 1 Owen Leonard 2016-02-11 16:39:24 UTC Comment hidden (obsolete)
Comment 2 Héctor Eduardo Castro Avalos 2016-02-11 17:12:00 UTC Comment hidden (obsolete)
Comment 3 Marc Véron 2016-02-11 17:24:06 UTC Comment hidden (obsolete)
Comment 4 Marc Véron 2016-02-11 17:28:33 UTC
Comment on attachment 47932 [details] [review]
Bug 15785 - Use Font Awesome icons in confirmation dialogs

Hector was faster :-) 
Obsoleting my sign-off patch.
Comment 5 Jonathan Druart 2016-02-12 13:09:05 UTC
Created attachment 47971 [details] [review]
Bug 15785 - Use Font Awesome icons in confirmation dialogs

We can simplify the staff client's CSS and reduce some image file usage
if we modify confirmation dialogs to use Font Awesome icons. This patch
makes this change for Administration templates.

This patch includes some whitespace changes, so please consider that
when looking at diffs.

To test, apply the patch and test deletion in the following cases. Test
both confirmation and cancel actions.

Administration -> Budgets -> Delete budget
Administration -> Funds -> Delete fund
Administration -> Authority types -> Delete authority type
Administration -> Authority types -> MARC Structure -> Delete tag
Administration -> Authority types -> MARC Structure -> Subfields -> Delete
Administration -> MARC bibliographic framework -> Delete framework
Administration -> MARC bibliographic framework -> MARC structure -> Delete
Administration -> Cities and Towns -> Delete city
Administration -> Classification sources -> Delete classification source
Administration -> Classification sources -> Delete classification filing rule
Administration -> Item types -> Delete
Administration -> Record matching rules -> Delete
Administration -> Patron attribute types -> Delete

Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
Works as advertised. Icons changed

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 6 Jonathan Druart 2016-02-12 13:09:35 UTC
I have fixed the conflicts with the revert of bug 13618
Comment 7 Jonathan Druart 2016-02-12 14:45:52 UTC Comment hidden (obsolete)
Comment 8 Héctor Eduardo Castro Avalos 2016-02-12 14:51:22 UTC
Hi someone attach a wrong bug number in this patch, please revert/obsolete the last patch attached
Comment 9 Jonathan Druart 2016-02-12 15:11:56 UTC
(In reply to Héctor Eduardo Castro Avalos from comment #8)
> Hi someone attach a wrong bug number in this patch, please revert/obsolete
> the last patch attached

Done
Comment 10 Brendan Gallagher 2016-02-23 20:47:03 UTC
Pushed to Master - Should be in the may 2016 release.  Thanks