D8291: correct Settings.isMobile usage

2017-10-27 Thread Marco Martin
This revision was automatically updated to reflect the committed changes. Closed by commit R169:162b877ad4b0: correct Settings.isMobile usage (authored by mart). REPOSITORY R169 Kirigami CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8291?vs=20724&id=21442 REVISION DETAIL https://

D8291: correct Settings.isMobile usage

2017-10-15 Thread Jan Mulder
janmulder added a comment. No commit access for me. Not needed, as I do not expect a huge amount of work on the repo. It is perfectly fine when mart commits it. REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D8291 To: janmulder, mart Cc: davidedmundson, plasma-deve

D8291: correct Settings.isMobile usage

2017-10-15 Thread David Edmundson
davidedmundson added a comment. Do you have commit access? REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D8291 To: janmulder, mart Cc: davidedmundson, plasma-devel, apol, mart, hein

D8291: correct Settings.isMobile usage

2017-10-15 Thread Marco Martin
mart accepted this revision. mart added a comment. This revision is now accepted and ready to land. thanks for spotting REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D8291 To: janmulder, mart Cc: plasma-devel, apol, davidedmundson, mart, hein

D8291: correct Settings.isMobile usage

2017-10-14 Thread Jan Mulder
janmulder created this revision. janmulder added a reviewer: mart. Restricted Application added a project: Kirigami. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Settings.isMobile is in the Kirigami import. Signed-off-by: Jan Mulder TEST PLAN Check that the