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

Hover picking over this surface is off #2208

Closed
etpinard opened this issue Dec 15, 2017 · 9 comments
Closed

Hover picking over this surface is off #2208

etpinard opened this issue Dec 15, 2017 · 9 comments
Assignees
Labels
bug something broken

Comments

@etpinard
Copy link
Contributor

etpinard commented Dec 15, 2017

First reported on https://community.plot.ly/t/3d-contained-surface-not-in-accordance-with-the-3d-containing-space/7327

See https://codepen.io/etpinard/pen/xpwGgV?editors=0010

peek 2017-12-15 10-48

@etpinard etpinard added the bug something broken label Dec 15, 2017
@etpinard
Copy link
Contributor Author

cc @archmoj this may be fixed by your work in #3281

@archmoj
Copy link
Contributor

archmoj commented Nov 26, 2018

This is a closed issue? It may not be addressed there.
@etpinard Anyway I updated your codepen to point to look at the branch here.

@archmoj archmoj self-assigned this Nov 26, 2018
@etpinard
Copy link
Contributor Author

Nop it's not a closed issue, unless it got resolved w/o us knowing. Just something I noticed while searching for another issue a couple minutes ago

@archmoj
Copy link
Contributor

archmoj commented Nov 26, 2018

Yesterday I was wondering also that it may not behave perfectly.
Then thanks for bringing this up : )

@archmoj
Copy link
Contributor

archmoj commented Nov 26, 2018

Now this was rather simple to fix.
@etpinard Please test it here.

@archmoj
Copy link
Contributor

archmoj commented Nov 28, 2018

Revised Demo with better fix on another branch.

@archmoj archmoj mentioned this issue Nov 30, 2018
@etpinard
Copy link
Contributor Author

@archmoj did #3299 fix this one?

@archmoj
Copy link
Contributor

archmoj commented Dec 17, 2018

@etpinard No it would be fixed with #3281. Here is the updated demo using latest code, in case you were interested to test.

@archmoj
Copy link
Contributor

archmoj commented Dec 18, 2018

Fixed in #3281.

@archmoj archmoj closed this as completed Dec 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something broken
Projects
None yet
Development

No branches or pull requests

2 participants