----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118972/#review61086 -----------------------------------------------------------
This review has been submitted with commit 18b6d854a01c010fd51db4977157802b0be0ea48 by Marco Martin to branch master. - Commit Hook On June 27, 2014, 1:09 p.m., Marco Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118972/ > ----------------------------------------------------------- > > (Updated June 27, 2014, 1:09 p.m.) > > > Review request for KDE Frameworks and Plasma. > > > Repository: plasma-framework > > > Description > ------- > > the color properties are remotely useful only when used from QML and risk to > explode in number in the future, it's not a good thing for a public c++ api. > > Also, they will become kinda deprecated as in largely replaced by the > ColorRole api. Too late to remove those properties completely, but still last > days to move them at least out of c++ api. (and in qml they could be removed > in the future in a 2.1 version of the import while still being present in 2.0 > if needed) > > This moves the color properties is a theme subclass available only from QML > (yes, it basically reintroduces the old themeproxy concept, but i don't think > it was completely a good move to move everything in theme) > > > Diffs > ----- > > src/declarativeimports/core/CMakeLists.txt e3d2f4e > src/declarativeimports/core/corebindingsplugin.cpp ebd9fd7 > src/declarativeimports/core/quicktheme.h PRE-CREATION > src/declarativeimports/core/quicktheme.cpp PRE-CREATION > src/plasma/theme.h 0211f73 > src/plasma/theme.cpp c193807 > > Diff: https://git.reviewboard.kde.org/r/118972/diff/ > > > Testing > ------- > > > Thanks, > > Marco Martin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
