Bug 7687

Summary: Remove problematic CSS gradient declarations targeting Internet Explorer
Product: Koha Reporter: Owen Leonard <oleonard>
Component: Browser compatibilityAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: minor    
Priority: P5 - low CC: chris, jcamins, paul.poulain
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 7687 - Remove problematic CSS gradient declarations targeting Internet Explorer
Bug 7687 - Remove problematic CSS gradient declarations targeting Internet Explorer
Bug 7687 - Remove problematic CSS gradient declarations targeting Internet Explorer

Description Owen Leonard 2012-03-08 20:27:49 UTC
Compatibility of CSS gradients in Internet Explorer is looking worse than I expected. I propose to remove the gradient from these elements in favor of a solid background:

- Cart and Lists buttons : Applying a background gradient to these elements
  causes the background to "bleed" out of the rounded corners in IE9 creating
  an ugly halo.

- Lists menu and patron menu items : These elements use gradients which in
  standards-compliant browsers accept several "steps" (color 1 blended with
  color 2 blended with color 3). IE only understands two colors. Better
  to have a solid background than the inferior fallback IE ends up with.
Comment 1 Owen Leonard 2012-10-30 18:17:39 UTC Comment hidden (obsolete)
Comment 2 Chris Cormack 2012-11-09 02:06:55 UTC Comment hidden (obsolete)
Comment 3 Paul Poulain 2012-11-29 16:59:46 UTC
QA comment
 * css only small patch
 * don't break anything on FF afaics

passed QA
Comment 4 Paul Poulain 2012-11-29 17:00:08 UTC
Created attachment 13779 [details] [review]
Bug 7687 - Remove problematic CSS gradient declarations targeting Internet Explorer

Removing and revising some CSS which causes problems in
Internet Explorer:

- User sidebar menu tabs in the OPAC
- Sidebar menu tabs in the staff client (circ, catalog, etc)
- List edit/delete buttons in the OPAC (when viewing the
  list of lists and viewing a list itself).

Also fixed is the sprite positioning for the "new list"
link which showed the wrong icon.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>

I tested this in Firefox, and Chromium and it didn't break anything,
I trust it works for IE too.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Comment 5 Jared Camins-Esakov 2012-11-30 03:18:32 UTC
This patch has been pushed to master.
Comment 6 Chris Cormack 2012-11-30 07:18:19 UTC
Pushed to 3.10.x, will be in 3.10.1