Summary: | Make RSS icon styleable | ||
---|---|---|---|
Product: | Koha | Reporter: | Robin Sheat <robin> |
Component: | OPAC | Assignee: | Robin Sheat <robin> |
Status: | CLOSED FIXED | QA Contact: | Jonathan Druart <jonathan.druart> |
Severity: | trivial | ||
Priority: | P5 - low | CC: | chris, jonathan.druart, paul.poulain, veron |
Version: | 3.8 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | Sponsored | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 8544 - make the RSS icon CSS-changeable
Bug 8544 - make the RSS icon CSS-changeable Bug 8544 - make the RSS icon CSS-changeable [SIGNED-OFF] Bug 8544 - make the RSS icon CSS-changeable |
Description
Robin Sheat
2012-07-31 20:28:49 UTC
Created attachment 11234 [details] [review] Bug 8544 - make the RSS icon CSS-changeable This gives the RSS bits on the search results some CSS classes so it can be modified with CSS or javascript. This applies on 3.8 nicely, should anyone want it for there too. Created attachment 11247 [details] [review] Bug 8544 - make the RSS icon CSS-changeable This gives the RSS bits on the search results some CSS classes so it can be modified with CSS or javascript. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Created attachment 11297 [details] [review] Bug 8544 - make the RSS icon CSS-changeable This gives the RSS bits on the search results some CSS classes so it can be modified with CSS or javascript. Updated: added also to the RSS icon shown when no results are found. I missed an instance of the RSS icon, so replacing the previous patch with one that does both. Created attachment 11320 [details] [review] [SIGNED-OFF] Bug 8544 - make the RSS icon CSS-changeable This gives the RSS bits on the search results some CSS classes so it can be modified with CSS or javascript. Updated: added also to the RSS icon shown when no results are found. Signed-off-by: Marc Veron <veron@veron.ch> Patch behaves as expected. Add 2 class for the RSS icon (the 2 only occurrences). Marking as Passed QA Patch pushed. It's a minor enhancement, probably not for 3.8, but chris_c, it's up to you, Robin put 3.8, I don't change it Adding a class to 2 html elements, no visible change at all, but allows them to be styled now. Pushed to 3.8.x, will be in 3.8.4 |