> On June 5, 2015, 11:12 a.m., Kai Uwe Broulik wrote: > > LGTM > > > > Thanks! It's also used by the wallpaper config. I guess the IconButton used > > by Kickoff and folderview could also be moved to KQC
>I guess the IconButton used by Kickoff and folderview could also be moved to >KQC Yes please. *hint* > On June 5, 2015, 11:12 a.m., Kai Uwe Broulik wrote: > > src/qmlcontrols/kquickcontrols/ColorButton.qml, line 54 > > <https://git.reviewboard.kde.org/r/124014/diff/1/?file=378857#file378857line54> > > > > You probably want to add an alias for the showAlphaChannel, title, and > > currentColor properties? For alpha I need to show it in the button too. The widget code creates a checkerboard pattern and blends it in appropriately, and frankly that sounds like effort. :( others makes sense. Will update. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124014/#review81225 ----------------------------------------------------------- On June 5, 2015, 10:46 a.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124014/ > ----------------------------------------------------------- > > (Updated June 5, 2015, 10:46 a.m.) > > > Review request for KDE Frameworks and Plasma. > > > Repository: kdeclarative > > > Description > ------- > > Add QQC clone of KColorButton. > > It's being used by SystemLoadViewer Plasmoid, Kai's clock review, and a mod > I'm doing for the notes plasmoid and it's something we'll want to have. > > High DPI support works perfectly as these are device independent pixels. > > > Diffs > ----- > > src/qmlcontrols/kquickcontrols/CMakeLists.txt > b7c08d7bcb46e53407d4b44fa593346f9d0a2137 > src/qmlcontrols/kquickcontrols/ColorButton.qml PRE-CREATION > src/qmlcontrols/kquickcontrols/qmldir > 31d8b145bf50491f94a14ba317b215fd6cc0ca32 > tests/colorButtonTest.qml PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/124014/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
