D24053: Properly handle ProcessAttribute enabled and default visibility

[Arjen Hiemstra]

ahiemstra created this revision.
ahiemstra added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ahiemstra requested review of this revision.

REVISION 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.

REPOSITORY
  R111 KSysguard Library

BRANCH
  plugin_enable

REVISION DETAIL
  https://phabricator.kde.org/D24053

AFFECTED FILES
  processcore/process_attribute.cpp
  processcore/process_attribute.h
  processui/ProcessModel.cpp
  processui/ProcessModel.h
  processui/ksysguardprocesslist.cpp

To: ahiemstra, #plasma
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190918/11d173c8/attachment.html>