ded / morpheus

A Brilliant Animator
505 stars 57 forks source link

Doesn't work with ems #6

Closed pyrsmk closed 13 years ago

pyrsmk commented 13 years ago

All is in the title :) I can't do animations based on ems and it's very important for responsive designs.

ded commented 13 years ago

sounds reasonable. i'll see what i can do.

ded commented 13 years ago

in the meantime - technically you can do tweens based on 'em' by using the generic tweener...

var el = document.getElementById('foo')
morpheus.tween(1000, function (pos) {
  el.style.fontSize = pos + 'em';
}, null, 100, 200);
pyrsmk commented 13 years ago

You're right, meanwhile I'll use tweens.

pyrsmk commented 13 years ago

Bizarrely, morpheus.tween is undefined.

ded commented 13 years ago

0.1.6 fixes the morpheus.tween

pyrsmk commented 13 years ago

Woh, really quick, thanks :)

ded commented 13 years ago

@pyrsmk — fyi: Morpheus has been updated to work with any CSS unit now. Check back on the main readme and see usage notes.

pyrsmk commented 13 years ago

Currently I can't test morpheus totally with ems. But with IE8 (in IETester), it returns me an "invalid call or argument" error.

ded commented 13 years ago

I'll have a look in the morning

ded commented 13 years ago

@pyrsmk - i'm not getting that issue. I have it working successfully in IE7,8,9 in native IE's. did you read the usage notes and make sure you're doing it right?

pyrsmk commented 13 years ago

I've no native IE but IETester ones. It means they are more chillies than the natives. My only native versions are IE9 and IE10, which work perfectly with morpheus ;)

ded commented 13 years ago

It's also working in IETester for me in IE6/7/8/9 (even 'em') — what does your code look like?

pyrsmk commented 13 years ago

It doesn't return me this anymore... Maybe it cames from another library... Respond I think. Sorry for the annoyance ;)