> 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? > > David Edmundson wrote: > 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 Edmundson wrote: > I'm being a noob. How can I extract the alpha value from a > http://doc.qt.io/qt-5/qml-color.html basic type?
Ooops. Found it. Sorry for the noise - 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
