feat(qbt): Show free space in status bar.

[userwiths]

Closes #335

I'm a frequent user of qbittorrent, so i can see how this would be useful.

The info about space left is send only on full_update and is contained in server_state.free_space_on_disk.

[tympanix]

Great addition, thanks 👍

Most clients will just show 0 for free disk space, which may be confusing.

[tympanix]

Might need an ng-if here for when this info is not available (which will be most clients, except qBittorrent)

[userwiths]

Ok, my first thought was to ty and add ng-if to the div, but after some trial and error I put in an ng-container around the div and put the condition on the container.

Tested with qBittorrent and Transmission. The new element was visible only in qBittorrent and was not showing in the other client.

The condition is based on client, because we save the free space in $scope and update it only if we receive info for free space again. So if it was ng-if="freeDiskSpace" and you had two servers (qBittorrent and some other) once you connect to qBit client you load that data, and when you switch to another client freeDiskSpace does not change.

Anyway, let me know if there is anything you think can be improved here.

[tympanix]

Thanks for the update.

I think is is important we keep the html agnostic to the client and not introduce conditions based on individual client.

What you probably should do instead is to fix the bug, that is, make sure to always update the 'diskSpaceLeft' variable to the appropriate value and set it to null or undefined when not available. Now you can reliably use this variable for an ng-if or similar.