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

IIssue with IE 11 #17

Closed
Chruse opened this issue Nov 6, 2015 · 6 comments · Fixed by #19
Closed

IIssue with IE 11 #17

Chruse opened this issue Nov 6, 2015 · 6 comments · Fixed by #19
Labels

Comments

@Chruse
Copy link

Chruse commented Nov 6, 2015

Hello!

I have the following issue. We have deployed the NipplesJS to the following software. Please see link: http://ec2-54-78-224-212.eu-west-1.compute.amazonaws.com/et/pano/show/1441792/separate On the right bottom corner there is deployed the nipplesJS.

The issue is that in IE11 there is no the so called Joystick ball (it doesn't show up) The Firefox and chrome are displaying joystick ball correctly. No problems with FF and Chrome.

If you debugg it you will see error Object doesn't support property or method 'contains'. You can debugg it yourself.

Please help, any suggestions what we have to do to make joystick ball to show up in IE 11.

Thanks in advance,

Christo

@Chruse
Copy link
Author

Chruse commented Nov 6, 2015

On additional comment. If you will click on link , please disable flash. The issue is in HTML5.

@yoannmoinet yoannmoinet added the bug label Nov 6, 2015
@yoannmoinet
Copy link
Owner

Thanks for reporting.

It seems that I'm using document.contains to detect if you've added the correct element to the dom.

I should have used document.body.contains because IE11 doesn't seem to shortcut it the same way as Chrome and FF.

Thanks for this, I'll fix it this week end.

If you need to fix it right now, I guess you could add something like
document.contains = document.body.contains.bind(document.body) when you're in IE11.
This will make the shortcut manually, while I'm fixing it.

@yoannmoinet
Copy link
Owner

Please also note bug #16 with the static configuration.

@yoannmoinet
Copy link
Owner

This should have been fixed.
Can you confirm it?

@Chruse
Copy link
Author

Chruse commented Nov 9, 2015

Hello Yaonn,

Thanks, issue solved.

One additional question.

If you will look at this link http://ec2-54-78-224-212.eu-west-1.compute.amazonaws.com/et/pano/show/1441792/separate (in case of IE) and use for the navigation virtual joystick (ball). Is it possible somewhere to change settings that the image movement from right > left or Left> right etc would be smoother. Currently it is moving like step by step. The image hangs and moves forward and after that hangs again and moves forward.

With flash the image movement is much more smoother.

Regards,

Christo Kuuse

From: Yoann Moinet [mailto:notifications@github.com]
Sent: Sunday, November 8, 2015 7:58 AM
To: yoannmoinet/nipplejs nipplejs@noreply.github.com
Cc: Chruse christo@ratus.ee
Subject: Re: [nipplejs] IIssue with IE 11 (#17)

This should have been fixed.
Can you confirm it?


Reply to this email directly or view it on GitHub #17 (comment) . https://github.com/notifications/beacon/AO9jSFIruD6vZ9JRVtA-6VPlqGsBX05Pks5pDtvZgaJpZM4GdYez.gif

@yoannmoinet
Copy link
Owner

I think I know why you have this behavior.

I you're listening to the move event, which is triggered only when your mouse is moving,
you're not constantly moving, this event isn't triggered all the time.
Only when changed.

In my opinion, you should have a constant movement applied, on which you'd apply a transformation value between :

  • -1 (left)
  • 0 (rest)
  • +1 (right)

And on move you'd change this transformation value based on the joystick position.

Though, I understand the need to have a constant triggering when active,
but it might not give a smoother experience than this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants