Page MenuHomePhabricator

Vector: IE9 renders the sidebar with extra separators
Closed, ResolvedPublic

Description

Author: jlemley

Description:
IE9 Rendering

The sidebar has extra separators displaying in IE9. They are part of the "portal expanded" div and specified under the "class=body" div. Other browsers recognize "background: none !important;" from ext.vector.collapsibleNav.css. IE seems not to see this when the code is minified. However, running with "?debug=true" causes proper rendering.

Screen shots courtesy of Krinkle.


Version: unspecified
Severity: normal

Attached:

IE9_Rendering.png (711×762 px, 220 KB)

Details

Reference
bz50516

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:57 AM
bzimport set Reference to bz50516.

jlemley wrote:

Chrome Rendering

Attached:

Chrome27_Rendering.png (681×742 px, 162 KB)

Though I can't confirm it right away, I think this is a recent-ish regression. I don't recall this bug from when I last used IE9 on a MediaWiki site with collapsibleNav enabled (from the Vector extension).

Is there any output in a JavaScript console (if IE has something like that)?

jlemley wrote:

Andre, Only this message: "SEC7115: :visited and :link styles can only differ by color. Some styles were not applied to :visited."

The console can be found in IE9 by pressing F12 and selecting the Console tab. Try it with "debug=true" to see a number of additional messages, though I'm pretty sure they are unrelated to this particular issue.

Bug 45536 suggests that this happened before and fixed itself magically.

  • Bug 45536 has been marked as a duplicate of this bug. ***

Change 71795 had a related patch set uploaded by Matmarex:
Use background-image: none instead of background: none

https://gerrit.wikimedia.org/r/71795

Apparently using background-image: none instead of background: none makes it work on IE. Could somebody verify the patch above? (I was relying on a browser screenshot service to debug this.)

jlemley wrote:

Confirmed the above patch with IE9. Thanks!

Change 71795 merged by jenkins-bot:
Use background-image: none instead of background: none

https://gerrit.wikimedia.org/r/71795

Well, this should be fixed now, then. The patch will be deployed to Wikimedia wikis according do the roadmap ([[mw:MediaWiki 1.22/Roadmap]]).

[Also adjusting product/component.]