<table><tr><td style="">rjvbb updated this revision to Diff 37188.<br />rjvbb marked 3 inline comments as done.<br />rjvbb edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-5ugyykw5a6qxgzr/">(Show Details)</a><br />rjvbb edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-bfy4hd55xdlthts/">(Show Details)</a><br />rjvbb added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D13899">View Revision</a></tr></table><br /><div><div><p>Patch updated as requested.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>The `readRGB()` method is named to indicate the fact it returns an RGB triplet via a QColor instance.<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>There's been a rumour that the `kdeglobals` store might disappear. If  and when that happens this code will have to be adapted but I don't see a reason to stop using the file now already.<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
<br />
</span></div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D13899?vs=37176&id=37188">https://phabricator.kde.org/D13899?vs=37176&id=37188</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13899">https://phabricator.kde.org/D13899</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/CMakeLists.txt<br />
src/kmessagewidget.cpp<br />
src/kthemesettings.cpp<br />
src/kthemesettings_p.h</div></div></div><br /><div><strong>To: </strong>rjvbb, Frameworks, VDG<br /><strong>Cc: </strong>broulik, kde-frameworks-devel, michaelh, crozbo, firef, ngraham, bruns, skadinna, aaronhoneycutt, mbohlender<br /></div>