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

Bubble series symbol doesn't reset to default when set to `null` #6792

Closed
hon2a opened this Issue Jun 5, 2017 · 2 comments

Comments

Projects
None yet
2 participants
@hon2a

hon2a commented Jun 5, 2017

I'm rendering a bubble chart with an enhancement - points with zero size are rendered using a custom "cross" symbol. The default for the series in the chart is to use the "circle" symbol. It's set up through plotOptions.bubble.marker.symbol. The point data for zero-size points looks like this:

{ x: 12, y: 34, z: 0, marker: { symbol: 'cross' } }

and for regular points like this:

{ x: 12, y: 34, z: 56, marker: { symbol: null } }

Expected behaviour

When I call .setData() on the series with updated points' sizes and symbols, the symbols should update (to crosses for points that now have zero size, back to circles for points that don't have zero size anymore).

Actual behaviour

When the points markers' symbols get updated from 'cross' to null, the symbol doesn't change.

Live demo with steps to reproduce

http://jsfiddle.net/hon2a/9x95qwqe/

(It doesn't show the scenario described above, but it clearly shows how the symbol doesn't change, but resetting e.g. color to null works just fine.)

Affected browser(s)

All.

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Jun 5, 2017

Contributor

Hi @hon2a

Thank you for reporting. You are right, symbol should change too. At this moment, it's necessary to set strict symbol, null won't work: http://jsfiddle.net/341qq3pc/

Contributor

pawelfus commented Jun 5, 2017

Hi @hon2a

Thank you for reporting. You are right, symbol should change too. At this moment, it's necessary to set strict symbol, null won't work: http://jsfiddle.net/341qq3pc/

@pawelfus pawelfus added the Bug label Jun 5, 2017

@pawelfus pawelfus self-assigned this Jun 5, 2017

pawelfus added a commit that referenced this issue Jun 5, 2017

@hon2a hon2a closed this Jun 5, 2017

@hon2a hon2a reopened this Jun 5, 2017

@hon2a

This comment has been minimized.

Show comment
Hide comment
@hon2a

hon2a Jun 5, 2017

Sorry, misclicked :)

Yeah, we're setting the default symbol explicitly for now. Luckily, we're just using circles in all other cases.

hon2a commented Jun 5, 2017

Sorry, misclicked :)

Yeah, we're setting the default symbol explicitly for now. Luckily, we're just using circles in all other cases.

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