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

Series (negative) color settings ignored on column hover #3636

Closed
mbalonso opened this Issue Dec 4, 2014 · 7 comments

Comments

Projects
None yet
4 participants
@mbalonso

mbalonso commented Dec 4, 2014

Given a column chart that has series w/negative values and a negativeColor setting, when you mouseover one of the columns w/a negative value, it changes the color of the column to the positive color.

Example: http://jsfiddle.net/malonso/7q97ao6y/1/

That fiddle was forked from http://jsfiddle.net/gh/get/jquery/1.9.1/highslide-software/highcharts.com/tree/master/samples/highcharts/demo/column-negative/ with the only difference being the addition of 'color' and 'negativeColor' attributes for a single series.

FWIW - This does NOT happen in 3.0.9 but does happen in 3.0.10 and 4.0.x.

ghost pushed a commit that referenced this issue Dec 5, 2014

@mbalonso

This comment has been minimized.

mbalonso commented Dec 5, 2014

Fantastic. Thanks @stephanevanraes

@mbalonso mbalonso closed this Dec 5, 2014

@Cederman

This comment has been minimized.

Cederman commented May 1, 2015

I do not mean to reopen this issue, but I notice the same behaviour for timeseries chart I just made using the current version. http://jsfiddle.net/0tn7vzwe/

@stannius

This comment has been minimized.

stannius commented Jul 7, 2015

This bug still reproduces for line charts as well. http://jsfiddle.net/wrow9fud/1/

@mbalonso

This comment has been minimized.

mbalonso commented Jul 9, 2015

@Cederman @stannius - I will try reopening this but if there is not any activity here in a day or so, it is probably worth opening a new issue as I doubt anyone is paying attention to this thread anymore.

@mbalonso mbalonso reopened this Jul 9, 2015

@pawelfus

This comment has been minimized.

Contributor

pawelfus commented Jul 9, 2015

We are trying to close duplicated topics, so reopening sounds very reasonable. With reopened ticket we also receive full history of changes.

@pawelfus

This comment has been minimized.

Contributor

pawelfus commented Sep 1, 2015

Internal note:
Fix should also consider all zones, not only positive/negative colors.

@pawelfus

This comment has been minimized.

Contributor

pawelfus commented Sep 1, 2015

Note: fix above works altogether with this commit.

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