Properly handle ProcessAttribute enabled and default visibility
ClosedPublic

Authored by ahiemstra on Wed, Sep 18, 2:08 PM.

Details

Summary

Rather than enabling them always, use column and application visibility
to determnine if an attribute should be enabled. In addition, use a
new visibleByDefault property to set the default visibility state of
columns.

Test Plan

Start ksysguard, see no plugin provided columns but they can be enabled
from the context menu.

Diff Detail

Repository
R111 KSysguard Library
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ahiemstra created this revision.Wed, Sep 18, 2:08 PM
Restricted Application added a project: Plasma. · View Herald TranscriptWed, Sep 18, 2:08 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ahiemstra requested review of this revision.Wed, Sep 18, 2:08 PM
davidedmundson accepted this revision.Wed, Sep 18, 2:21 PM
davidedmundson added a subscriber: davidedmundson.

Thanks!

This revision is now accepted and ready to land.Wed, Sep 18, 2:21 PM
This revision was automatically updated to reflect the committed changes.