Bug 410033 - Misleading label "current"
Summary: Misleading label "current"
Status: RESOLVED FIXED
Alias: None
Product: kinfocenter
Classification: Applications
Component: Energy Information (show other bugs)
Version: 5.14.5
Platform: Debian stable Linux
: NOR normal
Target Milestone: ---
Assignee: Kai Uwe Broulik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-07-20 12:12 UTC by Alois Wohlschlager
Modified: 2019-08-01 21:15 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 5.17.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alois Wohlschlager 2019-07-20 12:12:55 UTC
SUMMARY
In the "Energy Information" section, the current charge has the label "current". This is a bit misleading, because the word "current" also has the meaning of "electric current", which is not what this number is about. I suggest renaming the label to "Current charge".


STEPS TO REPRODUCE
1. Open KInfocenter
2. Go to the energy information tab

OBSERVED RESULT
The current charge has the label "current", leading to confusion with electric current.

EXPECTED RESULT
The label is not ambiguous.

SOFTWARE/OS VERSIONS
Windows: n/a
macOS: n/a
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.14.5
KDE Frameworks Version: 5.54.0
Qt Version: 5.11.3

ADDITIONAL INFORMATION
This can also be seen in more recent versions, the issue is also basically the same in French, but not in German. I do not know about other languages.
Comment 1 Kai Uwe Broulik 2019-07-22 07:15:12 UTC
Agreed. This looks like an easy fix, would you like to give it a go? :)
Comment 2 Alois Wohlschlager 2019-07-22 13:03:44 UTC
(In reply to Kai Uwe Broulik from comment #1)
> Agreed. This looks like an easy fix, would you like to give it a go? :)

Sure, but it's exam period and I am not very familiar with the KDE development process yet, so expect it to take a week or two,
Comment 3 Nate Graham 2019-07-23 00:03:57 UTC
I'd be happy to help!

First set up your development environment as detailed here: https://community.kde.org/Get_Involved/development

The relevant code is in the kinfocenter repo, and you can find the file in question by grepping for the string itself (\"Current\").

Once you've found the string and changed it in the git repo, you can submit a patch. Here's the documentation for how to do that: https://community.kde.org/Infrastructure/Phabricator

Let me know if you need a hand! Just email me. The process might seem tedious and scary, but hopefully it's not so bad, and there's plenty of help if you get stuck.
Comment 4 Nate Graham 2019-08-01 21:02:56 UTC
Git commit d5268f926798c2e7a19eb34afac46f7584fea7a8 by Nate Graham, on behalf of Alois Wohlschlager.
Committed on 01/08/2019 at 21:02.
Pushed by ngraham into branch 'master'.

Fix ambiguous "Current" label in energy information

Summary:
Replace the label text with "Current energy", to prevent confusion with electric current.

Reviewers: #plasma, #vdg, broulik, filipf, ngraham

Reviewed By: #vdg, filipf, ngraham

Subscribers: aspotashev, ngraham, plasma-devel

Tags: #plasma

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

M  +1    -1    Modules/energy/package/contents/ui/main.qml

https://commits.kde.org/kinfocenter/d5268f926798c2e7a19eb34afac46f7584fea7a8