<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>453922</bug_id>
          
          <creation_ts>2022-05-17 05:50:22 +0000</creation_ts>
          <short_desc>Decimal and thousands separators in ordinate axis labels are missing</short_desc>
          <delta_ts>2022-05-17 06:00:43 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>kmymoney</product>
          <component>reports</component>
          <version>unspecified</version>
          <rep_platform>Other</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>wishlist</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Thomas Baumgart">tbaumgart</reporter>
          <assigned_to name="KMyMoney Devel Mailing List">kmymoney-devel</assigned_to>
          
          
          <cf_commitlink>https://invent.kde.org/office/kmymoney/commit/2226d483f819dbee0036e4a84ac66509ca706203</cf_commitlink>
          <cf_versionfixedin>5.1.3</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2124040</commentid>
    <comment_count>0</comment_count>
    <who name="Thomas Baumgart">tbaumgart</who>
    <bug_when>2022-05-17 05:50:22 +0000</bug_when>
    <thetext>Source of this request: https://forum.kde.org/viewtopic.php?f=69&amp;t=174795 (though it bugs me too every now and then)

STEPS TO REPRODUCE
1. Open networth report (with networth &gt; 1000)

OBSERVED RESULT
Labels on y-axis do not contain thousand separator and are hard to read (e.g. 1000000). The same is valid for labels on the graph and tooltips.

EXPECTED RESULT
All values are displayed with thousand separators for better readability (e.g. 1.000.000 or 1,000,000 depending on the actual character used in the locale). Removing trailing decimal separator with only zeroes following is valid.

ADDITIONAL INFORMATION
Comment in current implementation contains the following comment:

    // convert label to double just to convert it back to string with desired precision
    // but without trailing zeros, separator, or decimal point

which was added with https://invent.kde.org/office/kmymoney/-/commit/5e4c8322 which fixes bug #420767</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2124041</commentid>
    <comment_count>1</comment_count>
    <who name="Thomas Baumgart">thb</who>
    <bug_when>2022-05-17 05:55:53 +0000</bug_when>
    <thetext>Git commit cbf8ad93429055d8b4059f0212b2afed687b7441 by Thomas Baumgart.
Committed on 17/05/2022 at 05:55.
Pushed by tbaumgart into branch &apos;master&apos;.

Format monetary values in graphs to contain separators

M  +6    -10   kmymoney/plugins/views/reports/core/kreportcartesianaxis.cpp
M  +20   -5    kmymoney/plugins/views/reports/core/kreportchartview.cpp

https://invent.kde.org/office/kmymoney/commit/cbf8ad93429055d8b4059f0212b2afed687b7441</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2124042</commentid>
    <comment_count>2</comment_count>
    <who name="Thomas Baumgart">thb</who>
    <bug_when>2022-05-17 06:00:43 +0000</bug_when>
    <thetext>Git commit 2226d483f819dbee0036e4a84ac66509ca706203 by Thomas Baumgart.
Committed on 17/05/2022 at 06:00.
Pushed by tbaumgart into branch &apos;5.1&apos;.

Format monetary values in graphs to contain separators
FIXED-IN: 5.1.3

(cherry picked from commit cbf8ad93429055d8b4059f0212b2afed687b7441)

M  +7    -11   kmymoney/plugins/views/reports/core/kreportcartesianaxis.cpp
M  +20   -5    kmymoney/plugins/views/reports/core/kreportchartview.cpp

https://invent.kde.org/office/kmymoney/commit/2226d483f819dbee0036e4a84ac66509ca706203</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>