Bug 23711

Summary: Icons on staff main page should be font icons
Product: Koha Reporter: Lucas Gass <lucas>
Component: Staff interfaceAssignee: Lucas Gass <lucas>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, gmcharlt
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
19.11.00
Bug Depends on:    
Bug Blocks: 23946    
Attachments: Bug 23711: Icons on staff main page should be font icons
Bug 23711: Icons on staff main page should be font icons
Bug 23711: Change icons and remove unused SVG
Screenshot
Bug 23711: Icons on staff main page should be font icons
Bug 23711: Change icons and remove unused SVG
Bug 23711: Icons on staff main page should be font icons
Bug 23711: Change icons and remove unused SVG
Bug 23711: Fix bad path
Bug 23711: Icons on staff main page should be font icons
Bug 23711: Change icons and remove unused SVG
Bug 23711: Fix bad path
Bug 23711: Icons on staff main page should be font icons
Bug 23711: Change icons and remove unused SVG
Bug 23711: Fix bad path

Description Lucas Gass 2019-10-01 12:47:40 UTC
On the staff main page we should use font icons (Font Awesome) instead of sprite.png. These are easier to adjust/customize with CSS than the sprites we currently use.
Comment 1 Lucas Gass 2019-10-01 13:01:35 UTC
Created attachment 93364 [details] [review]
Bug 23711: Icons on staff main page should be font icons

TEST PLAN: 
1. GO to '/cgi-bin/koha/mainpage.pl' and observe the icons. 
2. Apply the patch and clear the browser cache.
3. Refresh '/cgi-bin/koha/mainpage.pl' and make sure all the icons are still display. Note that some are not exactly the same as the old ones, but should be similar. Make sure you enable Course reserves and ILL so you can observe those icons as well.
Comment 2 Owen Leonard 2019-10-01 15:30:53 UTC
I'm curious why you didn't use 'fa-tag' for cataloging? And 'fa-gift' for acquisitions?

Also: misc/interface_customization/staff-home-icons-sprite.svg can be removed too.
Comment 3 Lucas Gass 2019-10-02 07:14:31 UTC
Created attachment 93395 [details] [review]
Bug 23711: Icons on staff main page should be font icons

TEST PLAN: 
1. GO to '/cgi-bin/koha/mainpage.pl' and observe the icons. 
2. Apply the patch and clear the browser cache.
3. Refresh '/cgi-bin/koha/mainpage.pl' and make sure all the icons are still display. Note that some are not exactly the same as the old ones, but should be similar. Make sure you enable Course reserves and ILL so you can observe those icons as well.
Comment 4 Lucas Gass 2019-10-02 07:14:37 UTC
Created attachment 93396 [details] [review]
Bug 23711: Change icons and remove unused SVG
Comment 5 Fridolin Somers 2019-10-02 09:49:58 UTC
Created attachment 93403 [details]
Screenshot
Comment 6 Fridolin Somers 2019-10-02 11:53:30 UTC
Created attachment 93439 [details] [review]
Bug 23711: Icons on staff main page should be font icons

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Comment 7 Fridolin Somers 2019-10-02 11:53:57 UTC
Created attachment 93440 [details] [review]
Bug 23711: Change icons and remove unused SVG

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Comment 8 Lucas Gass 2019-10-02 14:09:17 UTC
Created attachment 93479 [details] [review]
Bug 23711: Icons on staff main page should be font icons

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Comment 9 Lucas Gass 2019-10-02 14:09:28 UTC
Created attachment 93480 [details] [review]
Bug 23711: Change icons and remove unused SVG

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Comment 10 Lucas Gass 2019-10-02 14:09:39 UTC
Created attachment 93481 [details] [review]
Bug 23711: Fix bad path
Comment 11 Owen Leonard 2019-10-02 15:33:45 UTC
Created attachment 93496 [details] [review]
Bug 23711: Icons on staff main page should be font icons

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 12 Owen Leonard 2019-10-02 15:33:49 UTC
Created attachment 93497 [details] [review]
Bug 23711: Change icons and remove unused SVG

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 13 Owen Leonard 2019-10-02 15:33:54 UTC
Created attachment 93498 [details] [review]
Bug 23711: Fix bad path

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 14 Katrin Fischer 2019-10-03 07:21:02 UTC
Created attachment 93509 [details] [review]
Bug 23711: Icons on staff main page should be font icons

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 15 Katrin Fischer 2019-10-03 07:21:06 UTC
Created attachment 93510 [details] [review]
Bug 23711: Change icons and remove unused SVG

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 16 Katrin Fischer 2019-10-03 07:21:10 UTC
Created attachment 93511 [details] [review]
Bug 23711: Fix bad path

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 17 Martin Renvoize 2019-10-03 10:02:25 UTC
Nice work!

Pushed to master for 19.11.00