[KColorScheme] Add items to ColorSet and Role enums for the total number of…

Authored by ndavis on Dec 2 2019, 6:44 AM.

Description

[KColorScheme] Add items to ColorSet and Role enums for the total number of items

Summary:
I noticed that QPalette had these (see link below), 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 color sets or roles. For instance, a person might want to create an array of all foreground colors, but they would have to hardcode the array sizes without this patch.

I also removed some enum item value assignments that don't seem to have any use and might cause issues if new enum items are added.

https://code.qt.io/cgit/qt/qtbase.git/tree/src/gui/kernel/qpalette.h#n90

I plan to make a follow up patch to reduce the amount of hardcoded values in KColorScheme.

Test Plan: Open colorscheme editor in the Colors KCM to see if any colors or effects look broken.

Reviewers: Frameworks, dfaure, davidedmundson, hpereiradacosta

Reviewed By: dfaure

Subscribers: kde-frameworks-devel

Tags: Frameworks

Differential Revision: https://phabricator.kde.org/D25676

Details

Committed
ndavisDec 2 2019, 11:36 PM
Reviewer
dfaure
Differential Revision
D25676: [KColorScheme] Add items to ColorSet and Role enums for the total number of items
Parents
R265:719569193c7d: Remove unused include
Branches
Unknown
Tags
Unknown