On July 2nd, 2011, 9:49 p.m., Oswald Buddenhagen wrote:

hmm. but now things are still done twice in a kde session, no?
what was wrong with the suggestion to notify qt that it should update "stuff"?

On July 3rd, 2011, 2:52 p.m., Aurélien Gâteau wrote:

createApplicationPalette() is indeed called twice when running on a KDE session, but it is not a regression introduced by this change so I think it is outside of the scope for now. I tried not doing anything in kdisplaySetPalette() and call qt_x11_apply_settings_in_all_apps() from the kcm as Olivier suggested, but that didn't work: the palette change was not propagated to the running application.

What worries me right now is that the text area of KWrite does not get updated at runtime. I thought it was due to the widget being custom, but it correctly updates itself without the patch.

On July 16th, 2011, 10:35 p.m., Aurélien Gâteau wrote:

Finally found time to do more testing. It turns out the behavior of KWrite text area is the same with or without the patch so it's not a regression. Therefore, I think the patch should go in.

On July 16th, 2011, 10:42 p.m., Thomas Lübking wrote:

Sh*t - i forgot that I wanted to comment on that: kate keeps own color schemes for the text area, they're completely unrelated to he rest of the system.
(since you need to configure syntax highlightning and don't want that to run into a conflict with the system palette de toujours)

So yes, that's not a regression for sure, sorry.

On July 17th, 2011, 6:43 a.m., Dominik Haumann wrote:

With regard to kwrite: It uses the system colors as long as they were never changed. Changed once, these system settings are overwritten. Hence, this is very likely a KatePart issue.

On July 17th, 2011, 9:27 p.m., Aurélien Gâteau wrote:

Oh. Thanks Thomas and Dominik, it suddenly makes more sense! If there is no other objection I'd like to merge this patch this week. Anyone against that?

I just merged the changes in. Unless I spot some obvious regressions, I plan to backport the patch in time for 4.7.1.

- Aurélien

On July 2nd, 2011, 9:19 p.m., Aurélien Gâteau wrote:

When a KDE application is running on GNOME it looks odd right now because it does not use the GNOME palette and fonts, contrary to Qt-only applications. Attached patch fixes this by relying on the platform plugin to set the correct palette and fonts if we are not running in a full KDE session.

Patch was suggested by Olivier Goffart.

# On KDE
- Run kwrite on KDE => KDE palette and fonts
- Change palette and fonts from System Settings => kwrite updates itself correctly

# On GNOME
- Run kwrite on GNOME => GNOME palette and fonts
- Change palette and fonts from GNOME Tweak Tool => palette gets applied, font does not for now