hpereiradacosta accepted this revision. hpereiradacosta added a comment. This revision is now accepted and ready to land.
In D26225#583803 <https://phabricator.kde.org/D26225#583803>, @ndavis wrote: > - Change newPenWidthFrameRadius to frameRadiusNewPenWidth > > This sounds slightly better to me and allows it to show up next to frameRadius when autocompleting. What about "frameRadiusForPenWidth" ? I think the "New" is misleading. If you agree, please change and then commit. The rest sounds good. REPOSITORY R31 Breeze BRANCH replace-hardcoded-2 (branched from master) REVISION DETAIL https://phabricator.kde.org/D26225 To: ndavis, #breeze, #plasma, hpereiradacosta, ngraham Cc: 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