----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106490/ -----------------------------------------------------------
Review request for Plasma and Marco Martin. Description ------- The device notifier is using model["Free Space( Text)"] to show information which is not always available. As per Qt documentation, QML models need their roles defined at the beginning, thus, when the plasmoid is loaded with no accessible drives on the system, those two roles are not created (plasma/generic/dataengines/soliddevice/soliddeviceengine.cpp), and the plasmoid will always show "undefined" for its whole lifetime. This behaviour isn't observed on Linux apparently, I guess because UDisks always reports some accessible (internal) drives. It might be the same on FreeBSD (which I run), but since my configuration is a bit weird (ZFS), I spotted the bug. I reported the problem to Marco Martin and he blogged about it - http://www.notmart.org/index.php/BlaBla/Note_about_QML_plasmoids_and_dat - confirming my analysis. Diffs ----- plasma/generic/applets/devicenotifier/package/contents/ui/DeviceItem.qml 7ac95e7 plasma/generic/applets/devicenotifier/package/contents/ui/devicenotifier.qml bf07b10 Diff: http://git.reviewboard.kde.org/r/106490/diff/ Testing ------- With the patch applied, the plasmoid shows space information. The changes were confirmed by Marco Martin in his blog post. Thanks, Alberto Villa
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
