Model names have plural form.
Details
Details
- Reviewers
davidedmundson - Group Reviewers
KWin - Commits
- R108:b91cfae3080b: [kcmkwin] Rename EffectModel to EffectsModel
Compiles.
Diff Detail
Diff Detail
- Repository
- R108 KWin
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 7886 Build 7904: arc lint + arc unit
Comment Actions
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.