Bug 9278 - Remove unused OPAC CSS file sanop.css
Summary: Remove unused OPAC CSS file sanop.css
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: master
Hardware: All All
: P5 - low minor (vote)
Assignee: Owen Leonard
QA Contact:
URL:
Keywords:
Depends on:
Blocks: 8108
  Show dependency treegraph
 
Reported: 2012-12-12 20:47 UTC by Owen Leonard
Modified: 2013-05-23 06:24 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 9278 - Remove unused OPAC CSS file sanop.css (63.84 KB, patch)
2012-12-12 20:52 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 9278 - Remove unused OPAC CSS file sanop.css (63.93 KB, patch)
2012-12-13 09:16 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 9278 - Remove unused OPAC CSS file sanop.css (64.04 KB, patch)
2012-12-21 14:56 UTC, Elliott Davis
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2012-12-12 20:47:14 UTC
Paul has confirmed that this file is no longer in use. It should be removed.
Comment 1 Owen Leonard 2012-12-12 20:52:21 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2012-12-13 09:16:57 UTC Comment hidden (obsolete)
Comment 3 Elliott Davis 2012-12-21 14:56:50 UTC
Created attachment 14233 [details] [review]
Bug 9278 - Remove unused OPAC CSS file sanop.css

This patch removes an unused CSS file, sanop.css, included in both OPAC
themes even though it is referenced by neither.

To test, apply the patch and listen for the unhappy protestations of
libraries whose OPACs now look different. If there are none, the
patch works.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
According to Paul's answer (on the mailing list), I sign off this
patch.

Signed-off-by: Elliott Davis <elliott@bywatersolutions.com>
Seems to work as advertised
Comment 4 Jared Camins-Esakov 2012-12-22 21:25:36 UTC
This patch has been pushed to master.
Comment 5 Chris Cormack 2012-12-26 19:50:32 UTC
Pushed to 3.10.x will be in 3.10.2
Comment 6 Chris Cormack 2012-12-26 19:58:07 UTC
Pushed to 3.8.x, will be in 3.8.9