Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modify status bar mod to be able to display 1 throttle.down as well #37

Closed
chros73 opened this issue Aug 8, 2016 · 0 comments
Closed

Comments

@chros73
Copy link
Owner

chros73 commented Aug 8, 2016

Modify status bar mod (#4) to be able to display 1 throttle.down as well, also change name of config directive.

Display values of 1 throttle.up and 1 throttle.down in the first part of status bar.

Currently the beginning of status bar looks like this:
[Throttle 500/1500 KB] [Rate: 441.6/981.3 KB]

Include the max limit of the throttle, the main upload rate and the upload rate of the throttle (in this order); and main download rate and the download rate of the throttle.

Possible cases look like these:

[Throttle 200 / 500 KB] [Rate 107.4 / 298.6 KB]
[Throttle 200(114) / 500 KB] [Rate 107.0(1.0|105.9) / 307.6 KB]
[Throttle 200 / 500(250) KB] [Rate 124.7 / 298.2(298.2|0.0) KB]
[Throttle 200(114) / 500(250) KB] [Rate 115.9(1.7|114.2) / 333.9(333.9|0.0) KB]
  • make the displayable throttle names configurable via config file: ui.status.throttle.up.name and ui.status.throttle.down.name
  • don't display this extra info in the following cases:
    • there isn't any throttle name as the config variables suggest or the given name is "NULL"
    • the throttle is not throttled (=0)
    • the global upload/download is not throttled (=0) (the throttle won't be taken into account in this case)

Refers to: #4 .

@chros73 chros73 added this to the 1.4.2-0.9.6 milestone Aug 8, 2016
@chros73 chros73 changed the title Modify status bar mod to be able to display 1 throttle.down as well. Modify status bar mod to be able to display 1 throttle.down as well Aug 8, 2016
chros73 pushed a commit that referenced this issue Aug 8, 2016
@chros73 chros73 closed this as completed Aug 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant