D18712: [kcmkwin] Rename EffectModel to EffectsModel
David Edmundson <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
davidedmundson accepted this revision. davidedmundson added a comment. This revision is now accepted and ready to land. > Model names have plural form. Not universally, even the base class is in the singular. I certainly wouldn't bother changing any other instances you find around the rest of Plasma. I don't think anyone could mistakenly think this is a model of just one effect, so it's one of those changes that doesn't really make a difference. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D18712 To: zzag, #kwin, davidedmundson Cc: davidedmundson, kwin, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart