Allow setting position marker color from skin

[Daniel Schürmann]

Currently it is painted in axis color on the waveform and in maker color on the overview.

[Daniel Schürmann (daschuer)]

The attached patch introduces the optional
<PlayPosColor></PlayPosColor>
fallback is AxisColor.

The patch fixes also the wrong BgColor in all Shade skins.

It fixes the color scheme support for AxisColor. Because of this the Axis is now hard to see in Outline Classic. So we need to tweak the Axis color for Outline.

This batch is the base for some other 1.11.0 fixes, so it is itself 1.11.0

[jus (jus)]

Is it really necessary to introduce the <PlayPosColor> key?
For the waveform overview we use <MarkerColor> to set the playposition color, so can't we just make <MarkerColor> to work on the regular waveforms too ? Currently all skins use the <MarkerColor> key in the waveform part of the skin.xml, but i can't see it does anything.

[jus (jus)]

After trying your patch from lp:1094791, i just realized that you introduced <PlayPosColor> for both the waveform and waveform overview. This makes my previous comment somehow superfluous.

Is <MarkerColor> now obsolete with your patches and can be deleted from skin.xml?

[jus (jus)]

As for the BgColor in the Shade skins: This is by purpose iirc is the background transparent if no <BgColor> & <BgPixmap> are present

[RJ Skerry-Ryan (rryan)]

1) <MarkerColor> is used as a fallback by all the WaveformRenderMark (hotcues) and WaveformRenderMarkRange (loops) on the waveform and overview. So if no <Color> is provided for a mark or markrange then it falls back on <MarkerColor> of the parent.

I think we should get rid of MarkerColor everywhere and make WaveformMark and WavformMarkRange both use <AxesColor> as the fallback if no <Color> is present.

2) m_qMarkerColor can now be removed from WOverview I believe.

3) Not sure I see the difference between AxesColor and PlayPosColor?

[Daniel Schürmann (daschuer)]

Thank you for review!

1) is tracked in Bug #1107163

2) Yes, this code should be removed.

3) AxesColor is the color of the static horizontal line and PlayPosColor is the color of the static vertical line on the Waveform and the cursor in the WOverview.

[Daniel Schürmann (daschuer)]

committed to lp:mixxx/1.11 revision 3681

[Swiftb0y (swiftb0y)]

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/6844