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

List:       kde-bugs-dist
Subject:    [Bug 70073] printing with kspread gives big black areas
From:       Inge Wallin <inge () lysator ! liu ! se>
Date:       2005-05-01 0:02:05
Message-ID: 20050501000205.30878.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=70073         




------- Additional Comments From inge lysator liu se  2005-05-01 02:01 -------
CVS commit by ingwa: 

Fix first half of 70073: printing with kspread gives big black areas.

The second half will follow soon.

CCBUGS: 70073


  M +0 -1      TODO   1.121
  M +9 -2      kspread_cell.cc   1.714


--- koffice/kspread/TODO  #1.120:1.121
 @ -72,5 +72,4  @
   + 100643: KSpread prints a completely empty page at the end of      1 DONE
             the document.
-        Duplicate bug.  See analysis below
   + 100992: left border of cells with text, which is wider than the   1 ----
             cell breaks printout

--- koffice/kspread/kspread_cell.cc  #1.713:1.714
 @ -3199,6 +3199,6  @ void KSpreadCell::paintCellBorders( QPai
                                     bool paintRight, bool paintBottom,
                                     bool paintLeft,  bool paintTop,
-                                    QPen & rightPen, QPen & bottomPen,
-                                    QPen & leftPen,  QPen & topPen )
+                                    QPen & _rightPen, QPen & _bottomPen,
+                                    QPen & _leftPen,  QPen & _topPen )
 {
   KSpreadDoc * doc = sheet()->doc();
 @ -3245,4 +3245,11  @ void KSpreadCell::paintCellBorders( QPai
   }
 
+  // Must create copies of these since otherwise the zoomIt()
+  // operation will be performed on them repeatedly.
+  QPen  leftPen( _leftPen );
+  QPen  rightPen( _rightPen );
+  QPen  topPen( _topPen );
+  QPen  bottomPen( _bottomPen );
+
   // Determine the pens that should be used for drawing
   // the borders.
[prev in list] [next in list] [prev in thread] [next in thread]