D29005: Show free space available in the status bar

[Méven Car]

meven added a comment.


  In D29005#652415 <https://phabricator.kde.org/D29005#652415>, @shubham wrote:
  
  > Can somebody tell the reason why KCapacity bar is only showing the text?
  
  
  Maybe the status bar is too narrow for it to fit its progressbar, compare to dolphin for instance.

INLINE COMMENTS

> mainWindow.cpp:540
>          i18n("No files.") :
> -        i18np("1 file", "%1 files",fileCount);
> +        i18np("1 file", "%1 Files",fileCount);
>  

Don't change this, we use lowercase in this context (And you would need to change the singular case otherwise too).

> mainWindow.cpp:546
> +    
> +    if(space.isValid())
> +    {

style: `if (space.isValid()) {`

REPOSITORY
  R352 Filelight

REVISION DETAIL
  https://phabricator.kde.org/D29005

To: shubham, sandsmark, ngraham, kfunk
Cc: meven, kde-utils-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20200420/15c50fbc/attachment.html>