Bug 18810

Summary: Update Font Awesome to 4.7.0
Product: Koha Reporter: Josef Moravec <josef.moravec>
Component: TemplatesAssignee: Josef Moravec <josef.moravec>
Status: CLOSED FIXED QA Contact: Kyle M Hall <kyle>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, kyle, veron
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 18810: Update Font Awesome to 4.7.0
Bug 18810: Update Font Awesome to 4.7.0
Bug 18810: Update Font Awesome to 4.7.0
Bug 18810: Update Font Awesome to 4.7.0

Description Josef Moravec 2017-06-15 12:03:40 UTC

    
Comment 1 Josef Moravec 2017-06-20 17:25:57 UTC
Created attachment 64458 [details] [review]
Bug 18810: Update Font Awesome to 4.7.0

Test plan:

0) Apply the patch
1) Edit a template and use any of the new icons, see
http://fontawesome.io/icons/
2) Verify that the added icon is shown
Comment 2 Marc Véron 2017-06-21 07:31:05 UTC
Testing using the following lines in syspref IntranetmainUserblock:

<i class="fa fa-address-book"></i> Adress book<br />
<i class="fa fa-address-card"></i> Adress card<br />
<i class="fa fa-envelope-open"></i> Envelope open<br />
<i class="fa fa-times-rectangle"></i> Times rectangle

...OK
Comment 3 Marc Véron 2017-06-21 07:33:08 UTC
Created attachment 64482 [details] [review]
Bug 18810: Update Font Awesome to 4.7.0

Test plan:

0) Apply the patch
1) Edit a template and use any of the new icons, see
http://fontawesome.io/icons/
2) Verify that the added icon is shown

For alternative test see comment #2. Works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>
Comment 4 Kyle M Hall 2017-09-01 13:50:08 UTC
Created attachment 66742 [details] [review]
Bug 18810: Update Font Awesome to 4.7.0

Test plan:

0) Apply the patch
1) Edit a template and use any of the new icons, see
http://fontawesome.io/icons/
2) Verify that the added icon is shown

For alternative test see comment #2. Works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 5 Kyle M Hall 2017-09-01 13:50:58 UTC
Created attachment 66743 [details] [review]
Bug 18810: Update Font Awesome to 4.7.0

Test plan:

0) Apply the patch
1) Edit a template and use any of the new icons, see
http://fontawesome.io/icons/
2) Verify that the added icon is shown

For alternative test see comment #2. Works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 6 Jonathan Druart 2017-09-19 14:50:58 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 7 Fridolin Somers 2017-10-23 14:48:32 UTC
Enhancement not pushed to 17.05.x