Browse Source

moved point color from fill to color to let the possibility to use the color for stroke via css

pull/1992/head
Giacomo Zinetti 8 years ago
parent
commit
c67ca83b15
  1. 3
      src/scss/point.scss
  2. 4
      src/shape.line.js

3
src/scss/point.scss

@ -1,3 +1,6 @@
.c3-circle {
fill: currentColor;
}
.c3-circle._expanded_ { .c3-circle._expanded_ {
stroke-width: 1px; stroke-width: 1px;
stroke: white; stroke: white;

4
src/shape.line.js

@ -307,7 +307,7 @@ c3_chart_internal_fn.updateCircle = function () {
$$.mainCircle.enter().append("circle") $$.mainCircle.enter().append("circle")
.attr("class", $$.classCircle.bind($$)) .attr("class", $$.classCircle.bind($$))
.attr("r", $$.pointR.bind($$)) .attr("r", $$.pointR.bind($$))
.style("fill", $$.color); .style("color", $$.color);
$$.mainCircle $$.mainCircle
.style("opacity", $$.initialOpacityForCircle.bind($$)); .style("opacity", $$.initialOpacityForCircle.bind($$));
$$.mainCircle.exit().remove(); $$.mainCircle.exit().remove();
@ -317,7 +317,7 @@ c3_chart_internal_fn.redrawCircle = function (cx, cy, withTransition) {
return [ return [
(withTransition ? this.mainCircle.transition(Math.random().toString()) : this.mainCircle) (withTransition ? this.mainCircle.transition(Math.random().toString()) : this.mainCircle)
.style('opacity', this.opacityForCircle.bind(this)) .style('opacity', this.opacityForCircle.bind(this))
.style("fill", this.color) .style("color", this.color)
.attr("cx", cx) .attr("cx", cx)
.attr("cy", cy), .attr("cy", cy),
(withTransition ? selectedCircles.transition(Math.random().toString()) : selectedCircles) (withTransition ? selectedCircles.transition(Math.random().toString()) : selectedCircles)

Loading…
Cancel
Save