D8870: Synchronize the component with the one in Kirigami

2017-11-20 Thread Aleix Pol Gonzalez
This revision was automatically updated to reflect the committed changes. Closed by commit R242:aa4aa72d6322: Synchronize the component with the one in Kirigami (authored by apol). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8870?vs=22545

D8870: Synchronize the component with the one in Kirigami

2017-11-20 Thread Marco Martin
mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION DETAIL https://phabricator.kde.org/D8870 To: apol, #plasma, mart Cc: broulik, plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-moh

D8870: Synchronize the component with the one in Kirigami

2017-11-17 Thread Aleix Pol Gonzalez
apol updated this revision to Diff 22544. apol added a comment. Search all frameworks together as components REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8870?vs=22531&id=22544 BRANCH master REVISION DETAIL https://phabricator.kde

D8870: Synchronize the component with the one in Kirigami

2017-11-17 Thread Aleix Pol Gonzalez
apol updated this revision to Diff 22545. apol added a comment. unrelated things REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8870?vs=22544&id=22545 BRANCH master REVISION DETAIL https://phabricator.kde.org/D8870 AFFECTED FILES

D8870: Synchronize the component with the one in Kirigami

2017-11-17 Thread Aleix Pol Gonzalez
apol added a comment. Yes, this works, the code in question was in SystemTray, it looks just fine there. Also I've been using Heading in Discover for months and never saw this issue. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8870 To: apol,

D8870: Synchronize the component with the one in Kirigami

2017-11-17 Thread Kai Uwe Broulik
broulik added a comment. Are you sure this results in the same "bottom gap" below the item? Perhaps `bottomPadding` (new in Qt 5.6) could help here? REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8870 To: apol, #plasma Cc: broulik, plasma-devel,

D8870: Synchronize the component with the one in Kirigami

2017-11-17 Thread Aleix Pol Gonzalez
apol created this revision. apol added a reviewer: Plasma. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY Using the height property together with paintedHeight had binding loops. REPOSITORY R242 P