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:

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).
TheDJ created this task.Jul 29 2013, 3:24 PM
TheDJ added a comment.Jul 29 2013, 3:32 PM

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