Summary: | Remove unused CSS files | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Templates | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | normal | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, henridamien, magnus |
Version: | 3.4 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: | Proposed fix |
Description
Owen Leonard
2010-09-20 20:08:22 UTC
Created attachment 3111 [details] [review] Proposed fix Patch leaves blue.css since some are still using it. This bug is mentioned in: Fix for Bug 5234 - Remove unused CSS files http://lists.koha-community.org/pipermail/koha-patches/2011-February/013799.html The patch removes the CSS files, and I can see no ill effects in the staff client. Grepping through all Koha files for the file names of the CSS files does not show them being used anywhere. Signing off. Pushed please test It doesn't look like this patch was actually pushed. The files are still there. It was pushed, it is in master commit id 8464104fb299d1b1d707c0a083a04dce7c108953 http://git.koha-community.org/gitweb/?p=koha.git;a=commit;h=8464104fb299d1b1d707c0a083a04dce7c108953 Maybe it wasn't cherry-picked to 3.4? Ian pointed out to me on IRC that it looks like the removed files came back when we converted to T:T Reapplied and pushed |