[KMessageWidget] Update stylesheet when palette changes
ClosedPublic

Authored by broulik on Wed, Jul 4, 1:36 PM.

Details

Summary

The widget is "semi-transparent" by mixing in window background color but wouldn't update it when the color changed at runtime.

Test Plan

Changed system colors, message widget look changed immediately

Diff Detail

Repository
R236 KWidgetsAddons
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.Wed, Jul 4, 1:36 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptWed, Jul 4, 1:36 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
broulik requested review of this revision.Wed, Jul 4, 1:36 PM
broulik edited the summary of this revision. (Show Details)
broulik edited the test plan for this revision. (Show Details)
broulik updated this revision to Diff 37144.Wed, Jul 4, 1:44 PM
  • Juggle it around a bit
rjvbb added a comment.Wed, Jul 4, 2:17 PM

Otherwise LGTM (I also checked the in-app palette change myself: seems to work fine).

src/kmessagewidget.cpp
57

Nitpick: shouldn't this be called setStyleSheet or something similar that indicates a bit better that it defines the stylesheet itself?

rjvbb accepted this revision.Wed, Jul 4, 2:17 PM
This revision is now accepted and ready to land.Wed, Jul 4, 2:17 PM
ngraham accepted this revision.Wed, Jul 4, 2:28 PM

Thanks, works as expected and I couldn't find any regressions. Code looks good.

broulik added inline comments.Thu, Jul 5, 7:33 AM
src/kmessagewidget.cpp
57

It doesn't take any arguments, so I explicitly did not call it set

Late to the game, but also +1 from me.