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

Unable to set size of bubble anylonger #133

Closed
dss010101 opened this issue Dec 6, 2014 · 5 comments
Closed

Unable to set size of bubble anylonger #133

dss010101 opened this issue Dec 6, 2014 · 5 comments
Labels

Comments

@dss010101
Copy link

according to the comments in this stackoverflow question: http://stackoverflow.com/questions/22836594/dimple-js-change-radius-of-circles

The issue was fixed. But i try it here: http://jsfiddle.net/T6ZDL/43/
it does not seem to work.

@johnkiernander
Copy link
Member

You had the wrong class in your selector: http://jsfiddle.net/T6ZDL/44/

@johnkiernander
Copy link
Member

sorry, my mouse went a bit haywire there!

@dss010101
Copy link
Author

Thank you john. Yes my selector was off. There is an issue with this
when a duration value is specified for chart.draw() however. To resolve i
still have to use the setTimeout and set the timeout to something like
duration + 100. see here: http://jsfiddle.net/dizzy0ny/T6ZDL/47/

This is a bit of an issue as i am also using the 'legend trick'
(disassociating the legend from the chart) to be able to toggle series.
When i do that, then the transition looks a bit ackward.

Interestingly, the series.radius property that is available in 2.1 solves
my issue i believe. the problem is i can not use 2.1 because of this issue:
#101

Wondering if there is a way to easily solve either of these issues on my
end.

Thanks

On Sun, Dec 7, 2014 at 11:30 AM, John Kiernander notifications@github.com
wrote:

You had the wrong class in your selector: http://jsfiddle.net/T6ZDL/44/


Reply to this email directly or view it on GitHub
#133 (comment)
.

@johnkiernander
Copy link
Member

You can pick up version 2.1.1 here, it's not an official release yet but it has a fix for issue 101: https://github.com/PMSI-AlignAlytics/dimple/blob/master/dist/dimple.v2.1.1.js

@dss010101
Copy link
Author

Thanks John. For now I'm referencing 2.1 and overriding with the
getOrdereeList method from 2.1.1. that seems to work

On Sun, Dec 7, 2014 at 7:02 PM, John Kiernander notifications@github.com
wrote:

You can pick up version 2.1.1 here, it's not an official release yet but
it has a fix for issue 101:
https://github.com/PMSI-AlignAlytics/dimple/blob/master/dist/dimple.v2.1.1.js


Reply to this email directly or view it on GitHub
#133 (comment)
.

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

No branches or pull requests

2 participants