Bug in examples for columns and bars - sorting of date series.

[mattlangeman]

The bar and column date series charts are sorting by the date string (day of week).

[techniq]

Awe, likely a regression from upgrading Layer Cake to 8.0.0 today...

https://github.com/mhkeller/layercake/releases/tag/v8.0.0

[mattlangeman]

Awe, likely a regression from upgrading Layer Cake to 8.0.0 today...

https://github.com/mhkeller/layercake/releases/tag/v8.0.0

Yes. Changed my local Layer Cake back to 7.6.0 and the issue date sorting issue disappeared.

[mattlangeman]

Wondering if these examples should be using scaleTime instead of scaleBand. That would fix the sorting issue, but doing so causes the bars to have width=0. I'm thinking changes are needed to createDimensionGetter.

A side benefit is that you could also set the number of ticks. Current the Axis uses the full domain for ticks if it is scaleBand and it gets a bit crowded.

layerchart/src/lib/components/Axis.svelte

Lines 36 to 40 in ed04294

	$: tickVals = Array.isArray(ticks)
	? ticks
	: isScaleBand(scale)
	? scale.domain()
	: scale.ticks(typeof ticks === 'function' ? ticks(scale) : ticks);

[techniq]

Possibly... I know using a band scale with dates isn't optimal (especially with gaps in the data).

Here is some Observable Plot examples, which has an interval option

• https://observablehq.com/plot/transforms/interval
• https://observablehq.com/@observablehq/plot-ordinal-bar-chart
• https://observablehq.com/@observablehq/plot-ordinal-scale-interval

I opened a LayerCake issue to discuss as well

[techniq]

@mattlangeman Btw, improving date handling with band/bar charts was something I was tracking in #21 as well

[techniq]

Updated to 8.0.1, but still seeing the issue (both in LayerChart, and the LayerCake REPL).

[techniq]

Fixed by Layer Cake 8.0.2, and released as 0.24.3.