[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: koffice/kchart/kdchart
From:       Karl-Heinz Zimmer <khz () kde ! org>
Date:       2005-04-07 9:43:22
Message-ID: 200504071143.22962 () postmaster ! bugcops ! org
[Download RAW message or body]

Hi Fredrik,

thank you for this suggestion!
I agree that extending the meaning of the parameter to also causing
truncation makes a lot of sense and I have adjusted our main code
(and the documentation) as well.

"totalLen" somehow sounds like a forced len, not like a 'desire4d' len,
that's right.

Cheers
Karl-Heinz
 
Am Donnerstag, 7. April 2005 02:47 schrieb Fredrik Edemar:
> CVS commit by fedemar:
>
> * If the totalLen property is >0, the text label should be truncated at the
> position of totalLen. * Fixed an error in the documentation:
> KDCHART_DATA_VALUE_AUTO_DIGITS-> AXIS_LABELS_AUTO_DIGITS.
>
> CCMAIL: khz@kde.org
>
>
>   M +2 -0      KDChartAxesPainter.cpp   1.19
>   M +1 -1      KDChartAxisParams.cpp   1.11
>
>
> --- koffice/kchart/kdchart/KDChartAxesPainter.cpp  #1.18:1.19
> @@ -3477,4 +3477,6 @@ QString KDChartAxesPainter::applyLabelsF
>      if( blockAlign )
>          sVal.prepend( prefix );
> +    if ( totalLen > 0 )
> +        sVal.truncate( totalLen );
>      return sVal;
>  }
>
> --- koffice/kchart/kdchart/KDChartAxisParams.cpp  #1.10:1.11
> @@ -1487,5 +1487,5 @@ void KDChartAxisParams::setAxisLabelsCol
>    \param digitsBehindComma The number of digits to show behind the comma,
>      to have this calculated automatically just use the default value
> -    KDCHART_DATA_VALUE_AUTO_DIGITS.
> +    AXIS_LABELS_AUTO_DIGITS.
>    \sa setAxisLabelsFormat, axisLabelsDivPow10, axisLabelsDigitsBehindComma
>    */

-- 
Karl-Heinz         <mailto:khz@indeview.org>        <mailto:khz@kde.org>
  Zimmer                I n d e V i e w                    K D E
  Föhren       Presentations Beyond Limitations     Conquer your Desktop
www.fiehr.de            www.indeview.org                www.kde.org


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic