Bug 413954

Summary: Needless precision in night colour coordinates
Product: [Applications] systemsettings Reporter: Karl Ove Hufthammer <karl>
Component: kcm_nightcolorAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: minor CC: kwin-bugs-null, nate
Priority: NOR Keywords: junior-jobs
Version: 5.17.2   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.19.0

Description Karl Ove Hufthammer 2019-11-08 18:27:34 UTC
SUMMARY
When one uses the ‘Detect Location’ feature in the night colour settings, the latitude and longitude is returned with *four* decimal digits (e.g. 60.3984). This corresponds to a precision of ~10 meters! This is far too many digits (the sunset/sunrise times don’t change much if you move ten meters). It makes people think the last digits matter, when they don’t matter at all.

The number of decimal digits returned and shown should be reduced to two (corresponding to ~1 km) or one. (Even for two decimal digits, the change in the sunset/sunrise time for a change in the last digit is only a few *seconds*.)


STEPS TO REPRODUCE
1. Open the night colour system settings.
2. Set the operation mode to ‘Location’.
3. Press ‘Detect Location’.

OBSERVED RESULT
The latitude and longitude is updated and shown with four decimal digits.

EXPECTED RESULT
The latitude and longitude should be updated and shown with two decimal digits (or just one).

SOFTWARE/OS VERSIONS
Operating System: openSUSE Tumbleweed 20191104
KDE Plasma Version: 5.17.2
KDE Frameworks Version: 5.63.0
Qt Version: 5.13.1
Kernel Version: 5.3.8-1-default
OS Type: 64-bit
Processors: 4 × Intel® Core™ i5-2500 CPU @ 3.30GHz
Memory: 15,6 GiB
Comment 1 Shubham 2020-04-19 05:59:03 UTC
Git commit 35e18420a76ab4f738a813e67c2e304834b60bf7 by Shubham Jangra.
Committed on 19/04/2020 at 05:58.
Pushed by shubham into branch 'master'.

[kcm_nightcolor] Set 2 decimal places precision for coordinates

Summary: BUG: 413954

Reviewers: ngraham, davidedmundson

Reviewed By: davidedmundson

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D28958

M  +1    -1    kcms/nightcolor/package/contents/ui/main.qml

https://commits.kde.org/plasma-desktop/35e18420a76ab4f738a813e67c2e304834b60bf7