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

Selecting pie slices in styled mode #6005

Closed
PaulDalek opened this issue Nov 21, 2016 · 4 comments
Closed

Selecting pie slices in styled mode #6005

PaulDalek opened this issue Nov 21, 2016 · 4 comments
Assignees

Comments

@PaulDalek
Copy link
Contributor

@PaulDalek PaulDalek commented Nov 21, 2016

Behaviour

Selecting any of the pie slices in styled mode changes its initial color to the color of the first point.

Live demos

http://jsfiddle.net/d_paul/5hdctfhe/

@oysteinmoseng oysteinmoseng added this to the 5.0.4 milestone Nov 21, 2016
@garybrain
Copy link

@garybrain garybrain commented Nov 22, 2016

Hi, as a workaround I disabled the following two CSS entries.

.highcharts-pie-series .highcharts-point-select {
/* fill: inherit;
stroke: inherit; */
}
.highcharts-point-select {
/fill: #cccccc;
stroke: #000000;
/
}

@TorsteinHonsi
Copy link
Collaborator

@TorsteinHonsi TorsteinHonsi commented Nov 25, 2016

Ok, by setting the selected pie color to inherit, the intention was to override the general point select rule, that set the color to gray.

However the fill didn't go back to .highcharts-color-n, but to the general series color.

Any idea of how we can simply disqualify the fill set for .highcharts-point-select?

@TorsteinHonsi TorsteinHonsi removed this from the 5.0.5 milestone Nov 29, 2016
@KacperMadej
Copy link
Contributor

@KacperMadej KacperMadej commented Jul 31, 2017

@TorsteinHonsi How about:

(or highcharts-series-pie only for pies)

  • change general point select rule selector to:
    .highcharts-series:not(.highcharts-color-by-point) .highcharts-point-select {

(or highcharts-series-pie only for pies)

Result: http://jsfiddle.net/df74rdgv/

@TorsteinHonsi
Copy link
Collaborator

@TorsteinHonsi TorsteinHonsi commented Aug 2, 2017

Yes, that works well! Will you commit the fix?

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

Successfully merging a pull request may close this issue.

None yet
6 participants
You can’t perform that action at this time.