D27038: [KMessageWidget] Allow the style to change our palette

This revision was automatically updated to reflect the committed changes. Closed by commit R236:d946fb71ea22: [KMessageWidget] Allow the style to change our palette (authored by davidre). REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE

D27038: [KMessageWidget] Allow the style to change our palette

davidre updated this revision to Diff 76408. davidre added a comment. Rebase REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27038?vs=74701=76408 BRANCH polish REVISION DETAIL https://phabricator.kde.org/D27038 AFFECTED FILES

D27038: [KMessageWidget] Allow the style to change our palette

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. Meh, right. I guess it doesn't matter that much. REPOSITORY R236 KWidgetsAddons BRANCH polish (branched from master) REVISION DETAIL https://phabricator.kde.org/D27038 To: davidre,

D27038: [KMessageWidget] Allow the style to change our palette

davidre added a comment. In D27038#616095 , @apol wrote: > But then now we are not refreshing the palette when it changes outside, no? > > How about adding a boolean value when it's already happening to prevent the infinite recursion?

D27038: [KMessageWidget] Allow the style to change our palette

apol added a comment. But then now we are not refreshing the palette when it changes outside, no? How about adding a boolean value when it's already happening to prevent the infinite recursion? REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D27038 To:

D27038: [KMessageWidget] Allow the style to change our palette

davidre added a comment. Ping REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D27038 To: davidre, #frameworks Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27038: [KMessageWidget] Allow the style to change our palette

davidre added a dependency: D27035: [KMessageWidget] Draw it with QPainter instead of using stylesheet. REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D27038 To: davidre, #frameworks Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D27038: [KMessageWidget] Allow the style to change our palette

davidre added a dependent revision: D27039: [KStyle] Set the color of KMessageWidgets to the correct one from the current color scheme. REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D27038 To: davidre, #frameworks Cc: kde-frameworks-devel, LeGast00n, GB_2,

D27038: [KMessageWidget] Allow the style to change our palette

davidre created this revision. davidre added a reviewer: Frameworks. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. davidre requested review of this revision. REVISION SUMMARY After we set our palette give the current QStyle a chance to give us a better