D25677: [KColorScheme/KStatefulBrush] Switch hardcoded numbers for enum items

Noah Davis noreply at phabricator.kde.org
Mon Dec 2 07:04:35 GMT 2019


ndavis created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
ndavis requested review of this revision.

REVISION SUMMARY
  This also replaces some for-loops with C++11 range based for-loops and switches for simpler if/else control blocks.

TEST PLAN
  Open colorscheme editor in the Colors KCM to see if any colors or effects look broken.

REPOSITORY
  R265 KConfigWidgets

BRANCH
  remove-hardcoded-values (branched from master)

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

AFFECTED FILES
  src/kcolorscheme.cpp

To: ndavis
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191202/b18eee67/attachment.html>


More information about the Kde-frameworks-devel mailing list