Skip to content

Conversation

@keizie
Copy link
Contributor

@keizie keizie commented Jul 18, 2025

Description

To fix #246, add free space label into bottom status bar

Changes made

  • mainwindowstatusbar.* - add label widget and set text with other status labels
  • serverstats.* - add freeSpace setter and getter
  • rpc.* - getServerStats query freeSpace too

Additional Information

  • serverStats used to relay freeSpace value to setText inside mainwindowstatusbar.cpp QObject::connect, as Rpc::getDownloadDirFreeSpace is coroutine. I don't know how to handle co_await inside QObject::connect.
  • It takes number of seconds until freeSpace number updated, after debug log shows free-space call succeed. I think I somewhat missed "update right away" code.

@keizie keizie requested a review from equeim November 18, 2025 22:37
@equeim equeim merged commit c94e13c into equeim:master Dec 29, 2025
25 of 26 checks passed
@equeim
Copy link
Owner

equeim commented Dec 29, 2025

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

show available space directly on the left side or at the bottom

2 participants