Skip to content

Commit

Permalink
CategoryAxis: update category axis from dataset labels
Browse files Browse the repository at this point in the history
  • Loading branch information
wirew0rm committed Sep 27, 2023
1 parent ee5fb7d commit 849e8d4
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 0 deletions.
9 changes: 9 additions & 0 deletions chartfx-chart/src/main/java/io/fair_acc/chartfx/Chart.java
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
import io.fair_acc.bench.MeasurementRecorder;
import io.fair_acc.chartfx.axes.Axis;
import io.fair_acc.chartfx.axes.spi.AbstractAxis;
import io.fair_acc.chartfx.axes.spi.CategoryAxis;
import io.fair_acc.chartfx.axes.spi.DefaultNumericAxis;
import io.fair_acc.chartfx.legend.Legend;
import io.fair_acc.chartfx.legend.spi.DefaultLegend;
Expand Down Expand Up @@ -515,6 +516,14 @@ protected void runPreLayout() {
}
}

if (getBitState().isDirty(ChartBits.DataSetData)) {
for (Axis axis : getAxes()) {
if (axis instanceof CategoryAxis categoryAxis) { // update category axes
getRenderers().stream().filter(renderer -> renderer.getAxes().contains(categoryAxis)).findFirst().flatMap(renderer -> renderer.getDatasets().stream().findFirst()).ifPresent(categoryAxis::updateCategories);

Check warning on line 522 in chartfx-chart/src/main/java/io/fair_acc/chartfx/Chart.java

View check run for this annotation

Codecov / codecov/patch

chartfx-chart/src/main/java/io/fair_acc/chartfx/Chart.java#L522

Added line #L522 was not covered by tests
}
}
}

for (ChartPlugin plugin : plugins) {
plugin.runPreLayout();
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ public void updateAxes() {
if (yAxis == null) {
yAxis = chart.getYAxis();
}
getAxes().setAll(xAxis, yAxis);
}

protected Axis ensureAxisInChart(Axis axis) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ public void updateAxes() {
if (zAxis == null) {
zAxis = ensureAxisInChart(createZAxis());
}
getAxes().add(zAxis);
}

@Override
Expand Down

0 comments on commit 849e8d4

Please sign in to comment.