Dynamically set actions position. #240

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

DevinClark commented Dec 6, 2013

No description provided.

Contributor

DevinClark commented Dec 18, 2013

Thoughts, @jacobtoye?

Owner

jacobtoye commented Dec 18, 2013

Sorry I hadn't yet gotten to this one.

Looks OK, I'll give it a test today. Will need to check to see if it support IE7, from memory I think it has problems with offsetTop.

Also could you please revert the second commit. You don't need to build the library.

Contributor

DevinClark commented Dec 19, 2013

No problems using IE11 in IE7 emulation mode. I don't have a native IE7 browser to test in because we support IE9+.

Owner

jacobtoye commented Dec 19, 2013

@DevinClark thanks for this, I've cherry picked your commit. 762e872

@jacobtoye jacobtoye closed this Dec 19, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment