D13237: Show action.main more prominently on the ToolBarApplicationHeader

Marco Martin noreply at phabricator.kde.org
Thu May 31 16:34:01 UTC 2018


mart requested changes to this revision.
mart added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> PrivateActionToolButton.qml:62
>  
> -    flat: true
> +    palette.button: control.kirigamiAction ? control.kirigamiAction.icon.color : undefined
> +    property bool mainButton: false

note that this can't be used directly, as it depends from Qt 5.10
while if the button is colored by the method in D13232 <https://phabricator.kde.org/D13232> this shouldn't happen
so this shouldn't be pushed until can be done without breaking deps

REPOSITORY
  R169 Kirigami

REVISION DETAIL
  https://phabricator.kde.org/D13237

To: apol, #kirigami, mart, ngraham
Cc: ngraham, plasma-devel, apol, davidedmundson, mart, hein
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180531/8dea41c8/attachment.html>


More information about the Plasma-devel mailing list