/highcharts

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

point.states does not work in maps #6401

Closed
morganfree opened this Issue Feb 23, 2017 · 1 comment

Comments

Assignees

@pawelfus pawelfus

Labels
Projects
None yet
Milestone
No milestone
2 participants
@morganfree @pawelfus
@morganfree
Contributor

morganfree commented Feb 23, 2017

Actual behaviour

Setting point.states.hover.color has no effect on a hovered point.

Live demo with steps to reproduce

http://jsfiddle.net/z2q5jvmn/2/

  1. Hover over the point - color should be red according to the options

It stopped working in Highmaps 5, works correctly in Highmaps 4
http://jsfiddle.net/z2q5jvmn/3/
@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Feb 23, 2017

Contributor

Internal note:
It looks like an issue with column states (map series uses column prototype for states), demo: http://jsfiddle.net/5Le0h0d9/2/
Contributor

pawelfus commented Feb 23, 2017 •
edited

Internal note:
It looks like an issue with column states (map series uses column prototype for states), demo: http://jsfiddle.net/5Le0h0d9/2/

@pawelfus pawelfus self-assigned this Feb 23, 2017

@pawelfus pawelfus added Bug Regression and removed Bug labels Feb 23, 2017

pawelfus added a commit that referenced this issue Feb 23, 2017

@pawelfus
Fixed #6401, `point.states` didn't work for maps.
545f87d

pawelfus added a commit that referenced this issue Feb 24, 2017

@pawelfus
Addition to #6401, merge point states with series states.
2cfa665

@TorsteinHonsi TorsteinHonsi closed this in d00aad4 Feb 28, 2017

TorsteinHonsi added a commit that referenced this issue Feb 28, 2017

@pawelfus @TorsteinHonsi
Addition to #6401, merge point states with series states.
9118d4c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment