Bug 6706 - integrate crystal icon set
Summary: integrate crystal icon set
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) enhancement (vote)
Assignee: Nicole C. Engard
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-10 20:57 UTC by Nicole C. Engard
Modified: 2012-10-25 23:09 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 6706 : Add Crystal Clear Icon Set (801.91 KB, patch)
2011-08-10 21:05 UTC, Nicole C. Engard
Details | Diff | Splinter Review
follow up patch (1.88 KB, patch)
2011-08-12 13:46 UTC, Nicole C. Engard
Details | Diff | Splinter Review
Patches squashed, signed off (800.48 KB, patch)
2011-08-12 14:22 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nicole C. Engard 2011-08-10 20:57:13 UTC
The crystal icon set has a lot of images that will help the modern library. Patch forthcoming to integrate the set in to Koha.
Comment 1 Nicole C. Engard 2011-08-10 21:05:43 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2011-08-11 19:46:16 UTC
There are a couple of files in here which we don't want:

koha-tmpl/intranet-tmpl/prog/img/itemtypeimg/crystal-clear/.DS_Store
koha-tmpl/opac-tmpl/prog/itemtypeimg/crystal-clear/.DS_Store
Comment 3 Nicole C. Engard 2011-08-12 13:46:09 UTC Comment hidden (obsolete)
Comment 4 Owen Leonard 2011-08-12 14:22:16 UTC
Created attachment 4981 [details] [review]
Patches squashed, signed off
Comment 5 Ian Walls 2011-08-12 15:37:56 UTC
Can't use my 'easy to read' reason here, since there are 200 binary blobs going on, but this patch does just add a (very pretty) icon set, and the copyright notices.  No further action is required by Koha to pick up on this, so marking as Passed QA
Comment 6 Chris Cormack 2011-08-14 08:10:43 UTC
Pushed, works well, please test.

It is licensed under LGPL, which means we can distribute it under GPLv2 (as the license contains a clause allowing that)