Merge lp:~laney/ubuntu-system-settings/storageinfo-5.4 into lp:ubuntu-system-settings
Proposed by
Timo Jyrinki
on 2015-02-13
| Status: | Merged |
|---|---|
| Merged at revision: | 1312 |
| Proposed branch: | lp:~laney/ubuntu-system-settings/storageinfo-5.4 |
| Merge into: | lp:ubuntu-system-settings |
| Diff against target: |
43 lines (+11/-1) 3 files modified
plugins/about/PageComponent.qml (+1/-1) plugins/about/storageabout.cpp (+9/-0) plugins/about/storageabout.h (+1/-0) |
| To merge this branch: | bzr merge lp:~laney/ubuntu-system-settings/storageinfo-5.4 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Timo Jyrinki (community) | Approve on 2015-02-17 | ||
|
Review via email:
|
|||
To post a comment you must log in.
| Timo Jyrinki (timo-jyrinki) wrote : | # |
(plus for the actual bandaid made the item non-clickable because I needed to comment out the StorageInfo { } in PageComponent.qml as long as such QML class is not available)
| Timo Jyrinki (timo-jyrinki) wrote : | # |
Landed with the mentioned fixes together with Qt 5.4.
review:
Approve


Got functional as the bandaid solution by:
- Add #include <QtCore/ QStorageInfo> to plugins/ about/storageab out.cpp availableDiskSp ace", use "backendInfos. getFreeSpace" in plugins/ about/PageCompo nent.qml line 172
- Instead of "storageInfo.