From kde-commits  Wed Apr 30 22:31:57 2014
From: Dominik Haumann <dhaumann () kde ! org>
Date: Wed, 30 Apr 2014 22:31:57 +0000
To: kde-commits
Subject: [tikzkit] src/ui: cleanups
Message-Id: <E1Wfd2j-0007K6-D1 () scm ! kde ! org>
X-MARC-Message: https://marc.info/?l=kde-commits&m=139889712601057

Git commit c459e215b4b5fa91c7bc2a1fe518bfcfb3023590 by Dominik Haumann.
Committed on 30/04/2014 at 22:31.
Pushed by dhaumann into branch 'master'.

cleanups

M  +2    -0    src/ui/TikzView.cpp
M  +16   -16   src/ui/utils/Grid.cpp

http://commits.kde.org/tikzkit/c459e215b4b5fa91c7bc2a1fe518bfcfb3023590

diff --git a/src/ui/TikzView.cpp b/src/ui/TikzView.cpp
index 227ee54..b92a446 100644
--- a/src/ui/TikzView.cpp
+++ b/src/ui/TikzView.cpp
@@ -74,6 +74,8 @@ TikzView::TikzView(TikzDocument * doc, QWidget * parent)
     gridLayout->addWidget(viewport(), 1, 1);
 =

     setLayout(gridLayout);
+
+//     setViewportUpdateMode(FullViewportUpdate);
 }
 =

 TikzView::~TikzView()
diff --git a/src/ui/utils/Grid.cpp b/src/ui/utils/Grid.cpp
index 6365221..84e7f91 100644
--- a/src/ui/utils/Grid.cpp
+++ b/src/ui/utils/Grid.cpp
@@ -24,6 +24,9 @@
 #include <QGraphicsView>
 #include <QDebug>
 =

+// TODO (later): make this unit configurable
+static const tikz::Unit s_unit =3D tikz::Centimeter;
+
 namespace tikz {
 namespace ui {
 =

@@ -74,23 +77,20 @@ public:
             rect =3D r;
             zoom =3D z;
 =

-            // TODO (later): make this unit configurable
-            const tikz::Unit unit =3D tikz::Centimeter;
-
             // we want a line each 5 mm
-            const int lpu =3D linesPerUnit(unit);
-            qDebug() << "lines per unit" << lpu;
+            const int lpu =3D linesPerUnit(s_unit);
+//             qDebug() << "lines per s_unit" << lpu;
 =

             majorLines.clear();
             minorLines.clear();
 =

-            const Value one(1, unit);
+            const Value one(1, s_unit);
 =

-            tikz::Value left =3D tikz::Value(tikz::Value(rect.left()).conv=
ertTo(unit).value());
-            left =3D tikz::Value(std::floor(left.value()), unit);
+            tikz::Value left =3D tikz::Value(tikz::Value(rect.left()).conv=
ertTo(s_unit).value());
+            left =3D tikz::Value(std::floor(left.value()), s_unit);
 =

-            tikz::Value top =3D tikz::Value(tikz::Value(rect.top()).conver=
tTo(unit).value());
-            top =3D tikz::Value(std::ceil(top.value()), unit);
+            tikz::Value top =3D tikz::Value(tikz::Value(rect.top()).conver=
tTo(s_unit).value());
+            top =3D tikz::Value(std::ceil(top.value()), s_unit);
 =

             QVarLengthArray<QLineF, 100> lines;
             int i =3D 0;
@@ -133,7 +133,8 @@ void Grid::draw(QPainter * p, const QRectF & rect)
     d->updateCache(rect);
 =

     p->save();
-//     QPen pen(QColor(243, 243, 243));
+    p->setRenderHints(QPainter::Antialiasing, false);
+
     QPen pen(QColor(230, 230, 230));
     pen.setWidth(0);
     p->setPen(pen);
@@ -142,17 +143,16 @@ void Grid::draw(QPainter * p, const QRectF & rect)
     pen.setDashPattern(QVector<qreal>() << 5 << 5);
     p->setPen(pen);
     p->drawLines(d->minorLines.data(), d->minorLines.size());
+
     p->restore();
 }
 =

 tikz::Value Grid::snapValue(const tikz::Value & value) const
 {
-    // TODO (later): make this unit configurable
-    const tikz::Unit unit =3D tikz::Centimeter;
-    const int lpu =3D d->linesPerUnit(unit);
-    const Value one(1.0 / lpu, unit);
+    const int lpu =3D d->linesPerUnit(s_unit);
+    const Value one(1.0 / lpu, s_unit);
 =

-    const auto snappedValue =3D tikz::Value(qRound(value.convertTo(unit).v=
alue() / one.value()) * one.value(), unit).convertTo(value.unit());
+    const auto snappedValue =3D tikz::Value(qRound(value.convertTo(s_unit)=
.value() / one.value()) * one.value(), s_unit).convertTo(value.unit());
 //     qDebug() << value << "converts to:" << snappedValue;
     return snappedValue;
 }