ngraham requested changes to this revision.
ngraham added a comment.
This revision now requires changes to proceed.
Now the KCM doesn't load:
"file:///home/nate/kde/usr/share/kpackage/kcms/kcmaccess/contents/ui/main.qml"
"Error loading QML file.\n76: ListElement: cannot use script for property
value\n"
Also can you mark the inline comments as done once they're addressed? That
makes it easier to review. Thanks!
INLINE COMMENTS
> tcanabrava wrote in ModifierKeys.qml:51
> I can't correct you here, those where the original texts and since haven't
> got a clue on what they are talking about I choose not to change them.
> since haven't got a clue on what they are talking about
That's a great sign that they're in desperate need of being changed. :)
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D25375
To: tcanabrava, ngraham
Cc: ognarb, mart, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh,
fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai,
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra
