optional chart fill

[hercek]

optional chart fill

[dennisguse]

👍
Can you share a screenshot how it looks like?

[hercek]

Ok, here are some samples. The older version does not contain also the accepted PR which moves x-axis label lower right. So the comparison is not 100% fair. But I do not have only one commit older APK ready.

Portrait mode with chart fill

Portrait mode without chart fill

Landscape mode with chart fill

Landscape mode without chart fill

I think the problem is best visible in the landscape mode: try to follow the power curve in the interval from 4.2 km to 4.5 km (around the 3rd pin). It is more or less impossible in the version with the chart fill. Or I'm pretty much blind in my advanced age :)

Please specify whether you want version controlled by an option or just removing the fill all the time. Maybe it could be done with a 3-state checkbox - the 3-rd state would use fill for charts with only one or two data series.

[dennisguse]

Looks good!
I would skip an additional setting and hide the fill if at least three data series are shown.
Does this make sense?

[hercek]

Yes. I'll make a PR for this version: hiding fill when there are at least 3 data series visible.