Preset Browser colours must be string

[d.healey]

Setting the background colour or itemColour1 of the preset browser produces weird results if the colour isn't passed as a string, I think this is the only control that requires that.

[Christoph Hart]

Really? I've to check that, I thought it could be fed with both.

[d.healey]

Content.setHeight(300);

const var data1 = {"Type":"PresetBrowser", "ColourData":{bgColour:0xFF000000}};
const var p1 = Content.addFloatingTile("p1", 0, 0);
p1.set("width", 500);
p1.set("height", 150);
p1.setContentData(data1);

const var data2 = {"Type":"PresetBrowser", "ColourData":{bgColour:"0xFF000000"}};
const var p2 = Content.addFloatingTile("p2", 0, 150);
p2.set("width", 500);
p2.set("height", 150);
p2.setContentData(data2);

[Christoph Hart]

The upper one is better. Case closed :)

[Christoph Hart]

After a few hours of thinking about it and asking a few other people, I reconsider my opinion and decided to fix the bug:

- allowed FloatingTile JSON to use int values for ColourData · christophhart/HISE@18e6676

The open source framework for sample based instruments - - allowed FloatingTile JSON to use int values for ColourData · christophhart/HISE@18e6676

GitHub (github.com)

[d.healey]

Excellent thanks. It says Linux Build is failing, so I should hold off rebuilding?

[Christoph Hart]

Ah, no, that lousy Travis CI build file is broken since weeks. Things should be fine there (let me know if it doesn't).