Setting Default Colours

[AnthonySchultz]

In attempting to set default colours via setOptions() in the .config module the colours are not respected. I can, however, use Chart.defaults.global.colours. Looking at the code, in the getColours() it would seem that colours have precedence based on where they are set.

1. scope
2. Chart.defaults.global.colours
3. via setOptions.

To me this seems slightly out of order. I would think the would be

1. setOptions
2. scope
3. default

That way if someone needs to default all charts they can in one place, however, if colors are defined on the scope for an individual chart, they could be respected, and finally everything would fallback to the default colours.

What are your thoughts?

[jtblin]

Yeah makes sense to me as well. Can fix that later or feel free to send a PR.

[langan]

I've forked your repo and made the change here

langan@ae14e17

Not sure how I make a pull request

[jtblin]

Cool, click the green button on your fork above the file list and follow the instructions:

[langan]

Ok did that, let me know if that looks ok :)

[jtblin]

Thanks for the PR! Merged and published 0.7.1.