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

List:       kde-commits
Subject:    =?utf-8?q?=5Bcalligra=5D_plugins/textshape=3A_Even_better_soluti?=
From:       Sebastian Sauer <sebastian.sauer () kdab ! com>
Date:       2011-04-30 17:40:18
Message-ID: 20110430174018.912D3A60A9 () git ! kde ! org
[Download RAW message or body]

Git commit 81bfa3307da8a08ec4e88b16d63947c9fa2ef596 by Sebastian Sauer.
Committed on 30/04/2011 at 19:29.
Pushed by sebsauer into branch 'master'.

Even better solution for proper displaying the Frame Borders.

M  +3    -9    plugins/textshape/TextShape.cpp     

http://commits.kde.org/calligra/81bfa3307da8a08ec4e88b16d63947c9fa2ef596

diff --git a/plugins/textshape/TextShape.cpp b/plugins/textshape/TextShape.cpp
index 4a8dda1..df545fc 100644
--- a/plugins/textshape/TextShape.cpp
+++ b/plugins/textshape/TextShape.cpp
@@ -199,16 +199,10 @@ void TextShape::paintDecorations(QPainter &painter, const KoViewConverter &conve
             painter.setRenderHint(QPainter::Antialiasing);
 
         QPen pen(QColor(210, 210, 210)); // use cosmetic pen
-
-        QPointF tl(0.0, 0.0);
-        QRectF rect(tl, size());
-        painter.setPen(pen);
-        painter.drawLine(rect.topLeft(), rect.topRight());
-        painter.drawLine(rect.bottomLeft(), rect.bottomRight());
-
+        QPointF onePixel = converter.viewToDocument(QPointF(1.0, 1.0));
+        QRectF rect(QPointF(0.0, 0.0), size() - QSizeF(onePixel.x(), onePixel.y()));
         painter.setPen(pen);
-        painter.drawLine(rect.topRight(), rect.bottomRight());
-        painter.drawLine(rect.topLeft(), rect.bottomLeft());
+        painter.drawRect(rect);
         painter.restore();
     }
 }

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