[Network Monitor] Show correct bitrate
AbandonedPublic

Authored by ksmanis on May 1 2020, 12:36 PM.

Details

Reviewers
None
Group Reviewers
Plasma
Summary

Report the bitrate using SI prefixes, as is customary, instead of JEDEC
prefixes. Furthermore, make sure the reported bitrate respects the
user's locale when rounded.

Test Plan

Works as expected across different locales.

Diff Detail

Repository
R120 Plasma Workspace
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 26176
Build 26194: arc lint + arc unit
ksmanis created this revision.May 1 2020, 12:36 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 1 2020, 12:36 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ksmanis requested review of this revision.May 1 2020, 12:36 PM
mart added a subscriber: mart.May 4 2020, 9:55 AM

note that there is a whole rewrite pending for all systemmonitor applets in D28487 (and several related diffs)

Yeah, unfortunately I took notice after submitting the revision.

ksmanis abandoned this revision.May 17 2020, 10:34 AM