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

Color of null points on a map #5837

Closed
PaulDalek opened this Issue Oct 18, 2016 · 3 comments

Comments

Projects
None yet
4 participants
@PaulDalek
Contributor

PaulDalek commented Oct 18, 2016

Behaviour

Setting color for null points in styled mode through .highcharts-null-point class does not take effect on a map.

Live demo with steps to reproduce

http://jsfiddle.net/o6jz5svg/

@pawelfus pawelfus added the Highmaps label Oct 18, 2016

@oysteinmoseng

This comment has been minimized.

Show comment
Hide comment
@oysteinmoseng

oysteinmoseng Oct 18, 2016

Collaborator

Workaround: http://jsfiddle.net/o6jz5svg/1/

Internal note: seems like the null point class is not set on the points.

Collaborator

oysteinmoseng commented Oct 18, 2016

Workaround: http://jsfiddle.net/o6jz5svg/1/

Internal note: seems like the null point class is not set on the points.

@alvarezguille

This comment has been minimized.

Show comment
Hide comment
@alvarezguille

alvarezguille Mar 13, 2018

Contributor

Should the original live demo from this issue work?
It's getting highmaps without a version so should be latest code.

Contributor

alvarezguille commented Mar 13, 2018

Should the original live demo from this issue work?
It's getting highmaps without a version so should be latest code.

@oysteinmoseng

This comment has been minimized.

Show comment
Hide comment
@oysteinmoseng

oysteinmoseng Mar 13, 2018

Collaborator

Seems like a bug, it is getting the null point class (see http://jsfiddle.net/o6jz5svg/7/), but there is an additional fill style set. Reopening.

Collaborator

oysteinmoseng commented Mar 13, 2018

Seems like a bug, it is getting the null point class (see http://jsfiddle.net/o6jz5svg/7/), but there is an additional fill style set. Reopening.

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