Bug 447897

Summary: Add units with SI prefixes to Bit Rate and Sample Rate
Product: [Applications] Elisa Reporter: Jack Hill <jackhill3103>
Component: generalAssignee: Matthieu Gallien <matthieu_gallien>
Status: RESOLVED FIXED    
Severity: wishlist CC: nate
Priority: NOR    
Version: 21.12.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: https://invent.kde.org/multimedia/elisa/commit/be01acf1ca870cab159d0fc6e8fa1bafcd4deb6a Version Fixed In: 22.04
Attachments: Bit rate and sample rate not using SI units

Description Jack Hill 2022-01-03 20:59:42 UTC 
Created attachment 145092 [details]
Bit rate and sample rate not using SI units

STEPS TO REPRODUCE
1. Click the View Details button on a song, or open the Now Playing tab whilst playing a song
2. Look at the Bit Rate and Sample rate

OBSERVED RESULT
They show as a long numbers with no units, e.g. "320000" and "44100"

EXPECTED RESULT
They should use SI units, e.g. "320 kbps" and "44.1 kHz"

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: openSUSE Tumbleweed 20220101
KDE Plasma Version: 5.23.4
KDE Frameworks Version: 5.89.0
Qt Version: 5.15.2
Comment 1 Jack Hill 2022-04-03 22:20:33 UTC 
Git commit be01acf1ca870cab159d0fc6e8fa1bafcd4deb6a by Jack Hill.
Committed on 03/04/2022 at 21:11.
Pushed by ngraham into branch 'release/22.04'.

Add SI units to bit rate and sample rate

Format bit rate as "kbps" with no decimal places
Format sample rate as "kHz" with one decimal place

M  +6    -2    src/models/trackmetadatamodel.cpp

https://invent.kde.org/multimedia/elisa/commit/be01acf1ca870cab159d0fc6e8fa1bafcd4deb6a