KTurtle isn't released with Plasma, so that can't be the solution.

Sounds like it should be using KColorScheme to figure out these colors?

Also the commit message describes what it does too verbatim. Saying why you're doing it and how it fixes it is usually more useful.

In D16649#353684, @apol wrote:

KTurtle isn't released with Plasma, so that can't be the solution.

Sounds like it should be using KColorScheme to figure out these colors?

Also the commit message describes what it does too verbatim. Saying why you're doing it and how it fixes it is usually more useful.

Thanks for the suggestions. I'm curious to know why it won't work using QColors?

I'm not sure I understand the current problem, hence asking for a screenshot. It sounds to me like you're changing a colour for another because the current value doesn't fit your configuration, so it makes sense to query the configuration rather than just using a random other colour like we used to.

In D16649#354212, @apol wrote:

I'm not sure I understand the current problem, hence asking for a screenshot. It sounds to me like you're changing a colour for another because the current value doesn't fit your configuration, so it makes sense to query the configuration rather than just using a random other colour like we used to.

It looks like we already found someone who is trying fix it, see https://phabricator.kde.org/D15518.

In D16649#354395, @cfeck wrote:

It looks like we already found someone who is trying fix it, see https://phabricator.kde.org/D15518.

So, should I discard the changes or let it be open till the other review gets closed?