Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

eg.animate unit test is broken on Samsung Galaxy S2 (android 2.3) #120

Closed
happyhj opened this issue Feb 12, 2016 · 4 comments
Closed

eg.animate unit test is broken on Samsung Galaxy S2 (android 2.3) #120

happyhj opened this issue Feb 12, 2016 · 4 comments
Assignees
Milestone

Comments

@happyhj
Copy link
Contributor

happyhj commented Feb 12, 2016

"Relative animate Test - +=rotate(-30deg) translate(10px, 50%)" is broken.
It seems like last call back function is not executing.

@happyhj happyhj changed the title eg.animate unit test broken on Samsung Galaxy S3 (android 2.3) eg.animate unit test is broken on Samsung Galaxy S2 (android 2.3) Feb 12, 2016
@sculove sculove added the bug label Mar 24, 2016
@mixed mixed modified the milestone: 1.2.0 Mar 24, 2016
@jongmoon
Copy link
Contributor

I can find the pattern when it reproduce this issue. on Galaxy S2 (Android 2.3)

If current element's transform style has combination of matrix and exponential number. this issue occurs

for example, current element has

<div style="transform:matrix(-1.11022e-16, -1, 1, -1.11022e-16, 0, 0)"></div>

This element can not be transformed relatively

Below code have no UI change.
$el.css("transform":"matrix(-1.11022e-16, -1, 1, -1.11022e-16, 0, 0) translateX(10px)");

Solution?
Galaxy S2& Android 2.3 ==> Make exponetial number to normal number.

for exmple
transform:matrix(-0.00000000000000011102, -1, 1, -0.00000000000000011102, 0, 0)

But... I think it looks bad. Do we must support this?

@jongmoon
Copy link
Contributor

Resolved by #166

@jongmoon
Copy link
Contributor

It was not resolved by #166

@sculove
Copy link
Contributor

sculove commented Aug 3, 2017

Issue moved to naver/egjs-transform #10 via ZenHub

@sculove sculove closed this as completed Aug 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants