D11088: [KFilePlacesView] Call KDiskFreeSpaceInfo only if neccessary

2018-03-12 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R241:cbedf15311aa: [KFilePlacesView] Call KDiskFreeSpaceInfo only if neccessary (authored by broulik). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11088?vs=29085&id=29314

D11088: [KFilePlacesView] Call KDiskFreeSpaceInfo only if neccessary

2018-03-09 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D11088 To: broulik, #frameworks, davidedmundson Cc: ngraham, michaelh

D11088: [KFilePlacesView] Call KDiskFreeSpaceInfo only if neccessary

2018-03-09 Thread Kai Uwe Broulik
broulik updated this revision to Diff 29085. broulik added a comment. - Bail out if contentsOpacity is zero, just because the device is mounted and visible doesn't mean it shows its capacity bar (only on hover it would) REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kd

D11088: [KFilePlacesView] Call KDiskFreeSpaceInfo only if neccessary

2018-03-06 Thread Kai Uwe Broulik
broulik updated this revision to Diff 28797. broulik added a comment. - Slightly optimize REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11088?vs=28794&id=28797 REVISION DETAIL https://phabricator.kde.org/D11088 AFFECTED FILES src/filewidgets/kfileplacesv

D11088: [KFilePlacesView] Call KDiskFreeSpaceInfo only if neccessary

2018-03-06 Thread Kai Uwe Broulik
broulik created this revision. broulik added a reviewer: Frameworks. Restricted Application added a project: Frameworks. broulik requested review of this revision. REVISION SUMMARY I found this was called 350 times for me when opening the file dialog since for every paint pass it would needless