Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

IE8 tween/morph clip:rect breaks #2236

Closed
ibolmo opened this Issue · 1 comment

1 participant

@ibolmo
Owner

(from Lighthouse)

Some notes first:
-I looked through the tickets and this might be related to another but I cannot tell. -I am using IE Tester, not the actual IE8, so I don't know if this duplicates in the original (I'm using a virtual machine for IE6 and am using IE9 on this machine) -I may not be using clip: rect as it is meant to be used with tween/morph, however, it still seems to just work in every other browser including IE6,7, and 9.

Here is a JS Fiddle I set up to demonstrate:
http://jsfiddle.net/versatil/CzWfB/

setStyle works
setStyles works
tween breaks (in IE8 of IETester)
morph also breaks.

Couldn't figure out where it's going wrong, but instead of returning the full value ("rect(...)") it is returning everything prior to the first space, if you make the clip value comma-based, i.e. "rect(#px,#px,#px,#px)" then I am not sure what goes on but it runs through setting the style many times.

My apologies in advance if I'm doing something wrong here or otherwise have wasted your time, just seems to be a bug to me so I'm reporting it.

@ibolmo
Owner

Confirmed. I'm guessing this is a Fx.CSS.Parser problem. Need to investigate further.

@ibolmo ibolmo referenced this issue from a commit
Commit has since been removed from the repository and is no longer available.
@ibolmo ibolmo referenced this issue from a commit in ibolmo/mootools-core
@ibolmo ibolmo Fixes #2236.
Turns out this wasn't a Fx.CSS.parser issue but more of a
Element.getStyle needed to clean the value from rect(a, b, c, d) to our
preferred value of rect(a b c d).

This solves IE8 issue of returning rect(a,b,c,d) and non-webkit
returning rect(a, b, c, d). Element.setStyle still accepts rect(a b c d)
or rect(a, b, c, d).

PASSED: IE6-9; FFx 3-5, 8, 10; Chrome latest; Safari 5; Opera 11
baff862
@ibolmo ibolmo referenced this issue
Open

Fixes #2236. #2276

@ibolmo ibolmo closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.