Bug 118683 - Borders on ViewImages don't print
Summary: Borders on ViewImages don't print
Status: RESOLVED FIXED
Alias: None
Product: kst
Classification: Applications
Component: general (show other bugs)
Version: 1.x
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: kst
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-20 04:52 UTC by Netterfield
Modified: 2005-12-20 05:23 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Netterfield 2005-12-20 04:52:58 UTC
Version:           1.2.0_devel (using KDE 3.4.2 Level "b" , SUSE 10.0)
Compiler:          Target: i586-suse-linux
OS:                Linux (i686) release 2.6.13-15.7-default

Create ViewImage from the toolbar
In layout mode, give it a border.  It has a border on the screen.  Print it out.  Now it doesn't.

It should.
Comment 1 George Staikos 2005-12-20 05:22:58 UTC
SVN commit 489906 by staikos:

make images print properly again
BUG: 118683


 M  +8 -8      kstviewpicture.cpp  


--- trunk/extragear/graphics/kst/kst/kstviewpicture.cpp #489905:489906
@@ -69,14 +69,14 @@
 
 void KstViewPicture::paint(KstPainter& p, const QRegion& bounds) {
   p.save();
-  if (p.type() != KstPainter::P_PRINT && p.type() != KstPainter::P_EXPORT) {
-    if (p.makingMask()) {
-      p.setRasterOp(Qt::SetROP);
-      KstBorderedViewObject::paint(p, bounds);
-    } else {
+  if (p.makingMask()) {
+    p.setRasterOp(Qt::SetROP);
+    KstBorderedViewObject::paint(p, bounds);
+  } else {
+    KstBorderedViewObject::paint(p, bounds);
+    // FIXME: inefficient
+    if (p.type() != KstPainter::P_PRINT && p.type() != KstPainter::P_EXPORT) {
       QRegion boundary = bounds & _lastClipRegion;
-      KstBorderedViewObject::paint(p, bounds);
-      // FIXME: inefficient
       for (KstViewObjectList::Iterator i = _children.begin(); i != _children.end(); ++i) {
         boundary -= (*i)->clipRegion();
       }
@@ -84,7 +84,7 @@
       p.setClipRegion(boundary);
     }
   }
-  
+
   if (_image.isNull()) {
     QRect r(_geom);
     r.setWidth(_geom.width() - 1);