----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124014/#review81233 -----------------------------------------------------------
for qstyle based stuff we are depending on the qt devicepixelratio var, right? src/qmlcontrols/kquickcontrols/ColorButton.qml (line 40) <https://git.reviewboard.kde.org/r/124014/#comment55630> this may look very different with different qstyles? (like oxygen that has huge margin/border area around buttons) - Marco Martin 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