Skip to content

Add real-time download speed indicator to the transfer box - #5302

Open
iManGaaX wants to merge 4 commits into
multitheftauto:masterfrom
iManGaaX:patch-4
Open

Add real-time download speed indicator to the transfer box#5302
iManGaaX wants to merge 4 commits into
multitheftauto:masterfrom
iManGaaX:patch-4

Conversation

@iManGaaX

@iManGaaX iManGaaX commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Introduces a real-time download speed calculation and display feature to the MTA transfer box UI. Players can now see their active resource and map download speeds formatted cleanly in MB/s, KB/s, or B/s alongside the progress percentages.

Motivation

Previously, the transfer box showed only downloaded and total sizes, leaving players without a clear indication of their actual download performance or whether a slow download had stalled. This enhancement provides immediate, transparent feedback on network throughput during asset transfers.

Test plan

  1. Compiled the MTA client source code locally.
  2. Connected to a test server with large resources or map assets to trigger the transfer box.
  3. Verified that the download speed is dynamically calculated and displayed in the window title.
  4. Confirmed that the speed properly scales between B/s, KB/s, and MB/s and remains stable.

Checklist

  • Your code should follow the coding guidelines.
  • Smaller pull requests are easier to review. If your pull request is beefy, your pull request should be reviewable commit-by-commit.
Screenshot 2026-09-01 004600

@Haxardous

Copy link
Copy Markdown
Member

Thank you for the PR.

I believe it would be cleaner to exclude the "Speed:" prefix before the actual download speed, as it is self-explanatory.

@iManGaaX

iManGaaX commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

Thank you for the PR.

I believe it would be cleaner to exclude the "Speed:" prefix before the actual download speed, as it is self-explanatory.

Thanks. i have now removed the word “Speed:”

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.

2 participants