D25676: Add items to ColorSet and Role enums for the total number of items

2019-12-01 Thread Noah Davis
ndavis edited the summary of this revision.

REPOSITORY
  R265 KConfigWidgets

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

To: ndavis, #frameworks
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns


D25676: Add items to ColorSet and Role enums for the total number of items

2019-12-01 Thread Noah Davis
ndavis updated this revision to Diff 70717.
ndavis added a comment.


  - Also do Effects enum

REPOSITORY
  R265 KConfigWidgets

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D25676?vs=70716=70717

BRANCH
  enum-item-counts (branched from master)

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

AFFECTED FILES
  src/kcolorscheme.cpp
  src/kcolorscheme.h

To: ndavis, #frameworks
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns


D25676: Add items to ColorSet and Role enums for the total number of items

2019-12-01 Thread Noah Davis
ndavis edited the summary of this revision.

REPOSITORY
  R265 KConfigWidgets

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

To: ndavis, #frameworks
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns


D25676: Add items to ColorSet and Role enums for the total number of items

2019-12-01 Thread Noah Davis
ndavis edited the summary of this revision.

REPOSITORY
  R265 KConfigWidgets

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

To: ndavis, #frameworks
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns


D25676: Add items to ColorSet and Role enums for the total number of items

2019-12-01 Thread Noah Davis
ndavis edited the summary of this revision.

REPOSITORY
  R265 KConfigWidgets

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

To: ndavis, #frameworks
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns


D25676: Add items to ColorSet and Role enums for the total number of items

2019-12-01 Thread Noah Davis
ndavis added a reviewer: Frameworks.

REPOSITORY
  R265 KConfigWidgets

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

To: ndavis, #frameworks
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns


D25676: Add items to ColorSet and Role enums for the total number of items

2019-12-01 Thread Noah Davis
ndavis created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
ndavis requested review of this revision.

REVISION SUMMARY
  I noticed that QPalette had these, but KColorScheme didn't. Since there is no 
function to get the number of items in an enum for C++, it seems like it would 
be useful to have a built-in way to get the total number of items. I will make 
a follow up patch to reduce the amount of hardcoded values in KColorScheme.

REPOSITORY
  R265 KConfigWidgets

BRANCH
  enum-item-counts (branched from master)

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

AFFECTED FILES
  src/kcolorscheme.h

To: ndavis
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns