----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119464/#review63169 -----------------------------------------------------------
src/declarativeimports/plasmacomponents/qml/ToolButton.qml <https://git.reviewboard.kde.org/r/119464/#comment43931> Just curious, any reason to not use ToolButton here? http://qt-project.org/doc/qt-5/qml-qtquick-controls-toolbutton.html - Andrew Lake On July 25, 2014, 1:47 p.m., Marco Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119464/ > ----------------------------------------------------------- > > (Updated July 25, 2014, 1:47 p.m.) > > > Review request for KDE Frameworks and Plasma. > > > Repository: plasma-framework > > > Description > ------- > > this ports ToolButton. > It's a different style than Button since it's way more complicated. > > The ugly thing here is that QtControls ToolButton doesn't have a flat > property, it should probably be upstreamed, at least for compatibility with > the qwidgets ToolButton > > > Diffs > ----- > > src/declarativeimports/plasmacomponents/qml/ToolButton.qml 6dcbf7b > src/declarativeimports/plasmacomponents/qml/styles/ButtonStyle.qml d92a662 > src/declarativeimports/plasmacomponents/qml/styles/ComboBoxStyle.qml > 20caef6 > src/declarativeimports/plasmacomponents/qml/styles/ToolButtonStyle.qml > PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/119464/diff/ > > > Testing > ------- > > > Thanks, > > Marco Martin > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel