Skip to content
This repository has been archived by the owner on Jul 29, 2019. It is now read-only.

Add the source data to Point3d objects in Graph3D #1884

Merged
merged 3 commits into from
Oct 17, 2016

Conversation

TRManderson
Copy link
Contributor

This allows more complex tooltips, using information sent from the server + the source object

@@ -589,6 +590,7 @@ Graph3d.prototype._getDataPoints = function (data) {
point.x = data[i][this.colX] || 0;
point.y = data[i][this.colY] || 0;
point.z = data[i][this.colZ] || 0;
point.source = data[i];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

About the naming: I personally would prefer point.data over point.source

@mojoaxel
Copy link
Member

@TRManderson Could you maybe extend the graph3d tooltips example to use the new data? In any case this new feature should be documented and explained here. Thanks!

@TRManderson
Copy link
Contributor Author

Updated to include docs and example change. I couldn't come up with a compelling extension to the tooltips example, but I think I demonstrated the functionality non-horribly.

}
else {
data.add({x:x, y:y, z: z});
data.add({x:x, y:y, z: z, extra: extra_content[(x*y) % extra_content.length]});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please change the ident every where from "tab" to "spaces". (I also don't like it, but ithe the projects standart.)

This allows more complex tooltips, using information sent from the server + the source object
Vim config was off, edited in sublime instead
@TRManderson
Copy link
Contributor Author

Sorry about that - dodgy vim config meant I was typing tabs when I thought I had spaces. I've rebased develop with the changes requested.

@mojoaxel mojoaxel merged commit 217299e into almende:develop Oct 17, 2016
@mojoaxel
Copy link
Member

@TRManderson Thanks!

@mojoaxel mojoaxel mentioned this pull request Oct 24, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants