D7257: Set the QtQuick Controls 1 style name explicitly

2017-08-15 Thread Rik Mills
rikmills added a comment. I don't think it has an effect here, but for completeness just in case, I should mention that both Neon and KDE have a patch to #startkde/startkde.cmake https://packaging.neon.kde.org/plasma/plasma-workspace.git/tree/debian/patches/kubuntu_startkde-qtpath.diff?h

D7257: Set the QtQuick Controls 1 style name explicitly

2017-08-15 Thread Rik Mills
rikmills added a comment. In https://phabricator.kde.org/D7257#136069, @fvogt wrote: > Does it work with `export QT_QUICK_CONTROLS_1_STYLE=Breeze`? > > I don't know how the style selection of Quick Controls work, maybe someone more knowledgeable in that area could chime in here.

D7257: Set the QtQuick Controls 1 style name explicitly

2017-08-15 Thread Fabian Vogt
fvogt added a comment. Does it work with `export QT_QUICK_CONTROLS_1_STYLE=Breeze`? I don't know how the style selection of Quick Controls work, maybe someone more knowledgeable in that area could chime in here. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kd

D7257: Set the QtQuick Controls 1 style name explicitly

2017-08-15 Thread Rik Mills
rikmills added a comment. I am seeing this in both Neon dev-unstable edition (Xenial base), and Kubuntu's CI packages (Artful base). Another user is also seeing it in the Kubuntu CI builds. In both I have created a new user and tested under that, which should eliminate any caching issue

D7257: Set the QtQuick Controls 1 style name explicitly

2017-08-15 Thread Fabian Vogt
fvogt added a comment. openQA found something that looks related as well, but unsetting does not help: https://openqa.opensuse.org/tests/467477#step/systemsettings5/5 I cannot reproduce the issue you have either. Maybe some kind of cache or leftovers from other styles? This change shoul

D7257: Set the QtQuick Controls 1 style name explicitly

2017-08-15 Thread Rik Mills
rikmills added a comment. Seems it is. Unsetting that env var and restarting plasmashell makes it go away. Reversing that again make the ugliness shown there come back. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D7257 To: fvogt, #plasma, mart C

D7257: Set the QtQuick Controls 1 style name explicitly

2017-08-15 Thread Rik Mills
rikmills added a comment. Is this a result of this change? F3863427: d0cb251738.png REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D7257 To: fvogt, #plasma, mart Cc: rikmills, mart, plasma-devel, ZrenBot, progwolff

D7257: Set the QtQuick Controls 1 style name explicitly

2017-08-14 Thread Fabian Vogt
This revision was automatically updated to reflect the committed changes. Closed by commit R120:053d54488cdd: Set the QtQuick Controls 1 style name explicitly (authored by fvogt). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7257?vs=18020&id=18130

D7257: Set the QtQuick Controls 1 style name explicitly

2017-08-14 Thread Marco Martin
mart accepted this revision. mart added a comment. This revision is now accepted and ready to land. thanks for looking into that REPOSITORY R120 Plasma Workspace BRANCH warnfix REVISION DETAIL https://phabricator.kde.org/D7257 To: fvogt, #plasma, mart Cc: mart, plasma-devel, ZrenBot,

D7257: Set the QtQuick Controls 1 style name explicitly

2017-08-11 Thread Fabian Vogt
fvogt created this revision. Restricted Application added a project: Plasma. REVISION SUMMARY It does explicitly what the comment above the export implies. That way this warning on every use of Quick Controls 1 is gone: WARNING: Cannot find style "org.kde.desktop" - fallback: "/usr/lib64