<table><tr><td style="">ngraham 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/D23415">View Revision</a></tr></table><br /><div><div><p>I've standardized on the descriptive mood now. But I have not reverted back to "Smart" because it's simply a bad name, and it doesn't make sense to keep it. It doesn't matter what the feature is called internally, and how long it's had that name is irrelevant if it's a bad name. And "Smart" is a bad name because unlike all the other window placement modes, it is a generic meaningless term whose name does not explain or hint at what it will do. User-visible strings are crafted for the benefit of the user, not the developer.</p>

<p>Also I don't think renaming the enums is a good idea:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">If the point is to retain naming consistency throughout the codebase, then we'd need to update the strings written to and read from the config files as well, which would necessitate a kconf update script, which seems like total overkill; no other KDE project imposes such a high bar for simple string changes</li>
<li class="remarkup-list-item">If, to avoid the above, we just rename the enums but not all occurrences in the codebase, then we haven't actually maintained consistency, and there's no point in doing any code renaming at all</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>BRANCH</strong><div><div>better-window-placement-text (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23415">https://phabricator.kde.org/D23415</a></div></div><br /><div><strong>To: </strong>ngraham, KWin, VDG, ndavis<br /><strong>Cc: </strong>zzag, ndavis, kwin, kde-doc-english, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, gennad, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, skadinna, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart<br /></div>