D11134: ToolbarApplicationHeader layout

2018-03-07 Thread Marco Martin
This revision was automatically updated to reflect the committed changes. Closed by commit R169:2e7ec021ad0e: ToolbarApplicationHeader layout (authored by mart). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D11134?vs=28960&id=28963#toc REPOSITORY R169 Kirigami CHANGES SINCE LAST UPDA

D11134: ToolbarApplicationHeader layout

2018-03-07 Thread Aleix Pol Gonzalez
apol accepted this revision. apol added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > Units.qml:101 > > -readonly property QtObject __styleItem: QtQuickControlsPrivate.StyleItem > {elementType: "frame" } > +//readonly property QtObject __styleItem:

D11134: ToolbarApplicationHeader layout

2018-03-07 Thread Nathaniel Graham
ngraham added a comment. +1, this is a huge visual improvement in my eyes. REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D11134 To: mart, #kirigami Cc: ngraham, plasma-devel, apol, davidedmundson, mart, hein

D11134: ToolbarApplicationHeader layout

2018-03-07 Thread Marco Martin
mart updated this revision to Diff 28960. mart edited the summary of this revision. mart added a comment. - paddings fixes REPOSITORY R169 Kirigami CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11134?vs=28956&id=28960 BRANCH phab/toolbartweak REVISION DETAIL https://phabric