<table><tr><td style="">arojas created this revision.<br />arojas added reviewers: davidedmundson, Plasma.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D7953" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Use QQuickStyle::setStyle instead of an environment variable to set the QQC2 desktop style. This prevent the style from being inherited by plasmashell (or krunner) child processes, which makes QGuiApplications crash. The QQC1 style still needs to be set via an environment variable, so we explicitely unset it for non-QApplications to prevent them from crashing.</p>

<p>This adds a new QtQuickControls dependency, I hope it's OK at this stage (as it's already a dependency of other Plasma repos)</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Run systemsettings5 -> it is correctly themed<br />
Run minuet (pure QQC2) -> it doesn't crash<br />
Run kalgebra-mobile (QQC1 + QQC2) -> it doesn't crash.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R135 Integration for Qt applications in Plasma</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7953" rel="noreferrer">https://phabricator.kde.org/D7953</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
src/platformtheme/CMakeLists.txt<br />
src/platformtheme/kdeplatformtheme.cpp</div></div></div><br /><div><strong>To: </strong>arojas, davidedmundson, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>