Fix systemtray configuration

Authored by carewolf on Jan 26 2020, 9:46 AM.



The property currentValue is conflicting making the qml file not run.

Diff Detail

R120 Plasma Workspace
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
carewolf created this revision.Jan 26 2020, 9:46 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 26 2020, 9:46 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
carewolf requested review of this revision.Jan 26 2020, 9:46 AM

Thanks for noticing! The currentValue property was introduced with QtQuick.Controls 2.14 (Qt 5.14). Very convenient, unfortunately Plasma 5.18 must support Qt 5.12.

BTW. QML import system is stupid (or I am :) ). Apparently, using import QtQuick.Controls 2.5 will not allow you to use currentValue, but it will still conflict...


Instead of removing currentValue change it's name. This property is used in KeySequenceItem


I won't work in Qt =< 5.13. Just rename currentValue to something not-conflicting.

carewolf updated this revision to Diff 74379.Jan 26 2020, 10:04 AM

Changed name of property instead

Look good for me, but someone else from KDE team needs to approve this.

davidedmundson accepted this revision.Jan 26 2020, 4:17 PM

@konrad you're part of the kde team now. Feel free to approve things you're comfortable with.

This revision is now accepted and ready to land.Jan 26 2020, 4:17 PM
This revision was automatically updated to reflect the committed changes.