Cosmetic GUI issue - Githubissues

Zoxc / crusader

A network throughput and latency tester.

Apache License 2.0

116 stars 8 forks source link

Cosmetic GUI issue #39

Closed moeller0 closed 2 months ago

moeller0 commented 3 months ago

On the Client (and Latency) tabs of the GUI it reads:

Latency sample rate: XXX milliseconds Throughput sample rate: XXX milliseconds

technically these are sampling periods not rates/frequencies.

richb-hanover commented 2 months ago

I second Sebastian's comment: The labels could be "Latency sample interval:" and "Throughput sample interval:"

richb-hanover commented 2 months ago

Another comment on labeling the GUI for your consideration:

"Both" (the label for the combined throughput numbers) could simply be "Combined"
"Total" (the label for the combined latency numbers) could also be "Combined"

Thanks

Zoxc commented 2 months ago

"Both" (the label for the combined throughput numbers) could simply be "Combined"

Would Summed be more precise?

"Total" (the label for the combined latency numbers) could also be "Combined"

Maybe Round-trip? as that's just the common RTT.

moeller0 commented 2 months ago

I know, not my bike-shed, but I like the following colors best ;)

"bi-directional" or "aggregate", but combined or summed are decent colors too ;)

Regarding latency RTT/Round trip makes sense, but then it becomes tempting to call the other "upload one-way-delay" and "download one-way-delay" respectively, but that is rather long

richb-hanover commented 2 months ago

I like any of these. Also, RTT and OWD are short, and well-accepted abbreviations.

Maybe there could be a new "Help" tab. For 0.1.0, clicking it could go to the Crusader Home Page on github to provide explanations.

(Gosh, it's easy for me to envision more work for other people...) Thanks for listening