<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added inline comments.<br />This revision now requires changes to proceed.
</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/D20906">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D20906#inline-117409">View Inline</a><span style="color: #4b4d51; font-weight: bold;">main.xml:13</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);">    <span style="color: #00702a"><entry</span> <span style="color: #354bb3">name=</span><span style="color: #766510">"use_<span class="bright">default</span>_font"</span> <span style="color: #354bb3">type=</span><span style="color: #766510">"Bool"</span><span style="color: #00702a">></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);">      <span style="color: #00702a"><default><span class="bright"></span></span><span class="bright">tru</span>e<span style="color: #00702a"></default></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #00702a"><entry</span> <span style="color: #354bb3">name=</span><span style="color: #766510">"use_<span class="bright">custom</span>_font"</span> <span style="color: #354bb3">type=</span><span style="color: #766510">"Bool"</span><span style="color: #00702a">></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">      <span style="color: #00702a"><default><span class="bright"></span></span><span class="bright">fals</span>e<span style="color: #00702a"></default></span>
</div><div style="padding: 0 8px; margin: 0 4px; ">    <span style="color: #00702a"></entry></span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">If you change the name of any properties in the config xml file, users who have altered the settings will unexpectedly get new values and yell at you (well, whoever triages bugs, i.e. me and Christoph). :)</p>

<p style="padding: 0; margin: 8px;">If you do this, you need to provide a kconf update script that adjusts people's settings. See <a href="https://techbase.kde.org/Development/Tools/Using_kconf_update" class="remarkup-link" target="_blank" rel="noreferrer">https://techbase.kde.org/Development/Tools/Using_kconf_update</a></p>

<p style="padding: 0; margin: 8px;">Alternatively, you can keep the existing config name and just change the logic in the UI so that it still reads and writes the data in the same way, it's just presented to the user differently.</p></div></div></div></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/D20906">https://phabricator.kde.org/D20906</a></div></div><br /><div><strong>To: </strong>GB_2, Plasma, VDG, ngraham<br /><strong>Cc: </strong>ngraham, VDG, plasma-devel, Plasma, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>