<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: Plasma.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />broulik requested review of this revision.
</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/D26162">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The Settings object is kind of a write-only state and doesn't signal property changes unless "live" (where it would listen to other settings changing which we don't want inside the KCM) or only upon save().<br />
This is arguably a questionable design decision on my side :-) but usually it's not a problem as the KCM controls will have whatever state the user will put them in but it falls apart in case we switch to another page and back without saving. This hack is good enough for stable branch anyway imho.</p>

<p>BUG: 414098<br />
FIXED-IN: 5.17.5</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Went to position selector, chose a position, went back. Radio button moved to the bottom position. Before that it would only do it when I hit save</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26162">https://phabricator.kde.org/D26162</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/notifications/package/contents/ui/main.qml</div></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>