[Kst] extragear/graphics/kst/src/libkstmath

George Staikos staikos at kde.org
Fri Jun 2 14:43:26 CEST 2006


On Thursday 01 June 2006 12:00, Andrew Walker wrote:

>  int KstPainter::lineWidthAdjustmentFactor() const {
> -  const QRect& w(window());
> -  /* For square windows, the line width is in units of 0.1% of window size
> */ -  /* For printing to Letter or A10, line width is ~in points */
> -  const int factor = (w.width() + w.height()) / 2000;
> +  int factor = 1;
> +
> +  if (type() == P_PRINT || type() == P_EXPORT) {
> +    const QRect& w(window());
> +
> +    factor = (w.width() + w.height()) / 800;

   Where does this number come from?  The previous algorithm was documented, 
and was even explicitly documented:

------------------------------------------------------------------------
r520834 | staikos | 2006-03-20 17:58:11 -0500 (Mon, 20 Mar 2006) | 2 lines

document the magic number and increment it by one


   Please document the new approach.  Thanks.

-- 
George Staikos
KDE Developer				http://www.kde.org/
Staikos Computing Services Inc.		http://www.staikos.net/


More information about the Kst mailing list