Otherwise we'll initialize with dumb values and refresh it when shown, which
isn't all that effective.
Details
Details
- Reviewers
- None
- Group Reviewers
Plasma Frameworks
Tested against the feedback kcm
Diff Detail
Diff Detail
- Repository
- R296 KDeclarative
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 20806 Build 20824: arc lint + arc unit
Comment Actions
I kinda see what you're saying, I don't understand why we currently do anything on showEvent()
But now we're going to call load() twice, which doesn't sound very optimal.
Comment Actions
that depends how heavy the load() implementation is...
both ways are not optimal..
this behavior comes from KCModule, where the module nevel loads itself and a load() must be called from outside.
here may be worth doing that even if it's two loads as would avoid a binding reevaluation. othe the other hand.. a config read more, hmmm