Summary: | Intranet header toplinks display white rather than blue in < IE8 | ||
---|---|---|---|
Product: | Koha | Reporter: | David Cook <dcook> |
Component: | Browser compatibility | Assignee: | David Cook <dcook> |
Status: | CLOSED FIXED | QA Contact: | Jonathan Druart <jonathan.druart> |
Severity: | major | ||
Priority: | P5 - low | CC: | chris, jonathan.druart, paul.poulain |
Version: | 3.8 | ||
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: |
Bug 8414 : Intranet header toplinks display white rather than blue in < IE8
Bug 8414 : Intranet header toplinks display white rather than blue in < IE8 [SIGNED-OFF] Bug 8414 - Intranet header toplinks display white rather than blue in < IE8 |
Description
David Cook
2012-07-11 01:22:11 UTC
Created attachment 10764 [details] [review] Bug 8414 : Intranet header toplinks display white rather than blue in < IE8 Patch removes an IE7 hash hack that makes the top links on the intranet display white rather than dark blue against the light blue background. The lack of contrast is an identified accessibility issue. I would prefer to see this patch remove the line rather than comment it out. Created attachment 10777 [details] [review] Bug 8414 : Intranet header toplinks display white rather than blue in < IE8 Removed color property for #header a.toplinks Created attachment 11055 [details] [review] [SIGNED-OFF] Bug 8414 - Intranet header toplinks display white rather than blue in < IE8 Removes IE7 hash hack which caused head toplinks to display white rather than dark blue against light blue background Signed-off-by: Owen Leonard <oleonard@myacpl.org> I don't have an installation of IE7 handy but this patch is clearly correct and does nothing to change the status quo in any modern browser. QA Comments: Nothing to add Marking as Passed QA. I have not a IE<8 (In reply to comment #5) > I don't have an installation of IE7 handy but this patch is clearly correct > and does nothing to change the status quo in any modern browser. Nothing to add Pushed to 3.8.x, will be in 3.8.4 |