Page MenuHomePhabricator

Flyout renders faulty in IE7
Closed, ResolvedPublic

Description

Screenshot of IE 7

I tried this on a fresh machine, and it's indeed somewhat broken. Seems it is styling the wrong "Preferences" element.


Version: master
Severity: normal

Attached:

Screen_Shot_2013-07-29_at_17.09.13.png (841×1 px, 262 KB)

Details

Reference
bz52225

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 2:00 AM
bzimport set Reference to bz52225.
bzimport added a subscriber: Unknown Object (MLST).

This seems caused by the cloning, and if you google a bit, you'll note that jquery has had a LOT of problems with changing cloned content in IE versions...

Probably best to avoid that.

Change 76615 had a related patch set uploaded by Bsitu:
(bug 52225) Some javascript and css fix for IE7

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

bsitu wrote:

(In reply to comment #1)

This seems caused by the cloning, and if you google a bit, you'll note that
jquery has had a LOT of problems with changing cloned content in IE
versions...

Probably best to avoid that.

You are right, the clone() doesn't seem to break the layout in IE7

bsitu wrote:

(In reply to comment #3)

(In reply to comment #1)

This seems caused by the cloning, and if you google a bit, you'll note that
jquery has had a LOT of problems with changing cloned content in IE
versions...

Probably best to avoid that.

You are right, the clone() doesn't seem to break the layout in IE7

Oops, typo, I meant "does seem to"

Change 76615 merged by jenkins-bot:
(bug 52225) Some javascript and css fix for IE7

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