Differential D11629 Diff 31706 lib/documentview/abstractimageview.cpp

Changeset View

Standalone View

View Options

lib/documentview/abstractimageview.cpp

Show All 24 Lines
25 25​
26// KDE 26​// KDE
27 27​
28// Qt 28​// Qt
29#include <QDebug> 29​#include <QDebug>
30#include <QCursor> 30​#include <QCursor>
31#include <QGraphicsSceneMouseEvent> 31​#include <QGraphicsSceneMouseEvent>
32#include <QStandardPaths> 32​#include <QStandardPaths>
33​#include <QPainter>
33 34​
34namespace Gwenview 35​namespace Gwenview
35{ 36​{
36 37​
37static const int UNIT_STEP = 16; 38​static const int UNIT_STEP = 16;
38 39​
39struct AbstractImageViewPrivate 40​struct AbstractImageViewPrivate
40{ 41​{
Show All 10 Lines
51 52​
52 qreal mZoom; 53​ qreal mZoom;
53 bool mZoomToFit; 54​ bool mZoomToFit;
54 bool mZoomToFill; 55​ bool mZoomToFill;
55 QPointF mImageOffset; 56​ QPointF mImageOffset;
56 QPointF mScrollPos; 57​ QPointF mScrollPos;
57 QPointF mLastDragPos; 58​ QPointF mLastDragPos;
58 59​
60​ const QPixmap mAlphaBackgroundTexture = createAlphaBackgroundTexture();
61​
59 void adjustImageOffset(Verbosity verbosity = Notify) 62​ void adjustImageOffset(Verbosity verbosity = Notify)
60 { 63​ {
61 QSizeF zoomedDocSize = q->documentSize() * mZoom; 64​ QSizeF zoomedDocSize = q->documentSize() * mZoom;
62 QSizeF viewSize = q->boundingRect().size(); 65​ QSizeF viewSize = q->boundingRect().size();
63 QPointF offset( 66​ QPointF offset(
64 qMax((viewSize.width() - zoomedDocSize.width()) / 2, qreal(0.)), 67​ qMax((viewSize.width() - zoomedDocSize.width()) / 2, qreal(0.)),
65 qMax((viewSize.height() - zoomedDocSize.height()) / 2, qreal(0.)) 68​ qMax((viewSize.height() - zoomedDocSize.height()) / 2, qreal(0.))
66 ); 69​ );
Show All 37 Lines
104 void setupZoomCursor() 107​ void setupZoomCursor()
105 { 108​ {
106 // We do not use "appdata" here because that does not work when this 109​ // We do not use "appdata" here because that does not work when this
107 // code is called from a KPart. 110​ // code is called from a KPart.
108 const QString path = QStandardPaths::locate(QStandardPaths::GenericDataLocation, QStringLiteral("gwenview/cursors/zoom.png")); 111​ const QString path = QStandardPaths::locate(QStandardPaths::GenericDataLocation, QStringLiteral("gwenview/cursors/zoom.png"));
109 QPixmap cursorPixmap = QPixmap(path); 112​ QPixmap cursorPixmap = QPixmap(path);
110 mZoomCursor = QCursor(cursorPixmap, 11, 11); 113​ mZoomCursor = QCursor(cursorPixmap, 11, 11);
111 } 114​ }
115​
116​ QPixmap createAlphaBackgroundTexture()
117​ {
118​ QPixmap pix = QPixmap(32, 32);
119​ QPainter painter(&pix);
120​ painter.fillRect(pix.rect(), QColor(128, 128, 128));
121​ const QColor light = QColor(192, 192, 192);
122​ painter.fillRect(0, 0, 16, 16, light);
123​ painter.fillRect(16, 16, 16, 16, light);
124​ return pix;
125​ }
112}; 126​};
113 127​
114AbstractImageView::AbstractImageView(QGraphicsItem* parent) 128​AbstractImageView::AbstractImageView(QGraphicsItem* parent)
115: QGraphicsWidget(parent) 129​: QGraphicsWidget(parent)
116, d(new AbstractImageViewPrivate) 130​, d(new AbstractImageViewPrivate)
117{ 131​{
118 d->q = this; 132​ d->q = this;
119 d->mControlKeyIsDown = false; 133​ d->mControlKeyIsDown = false;
120 d->mEnlargeSmallerImages = false; 134​ d->mEnlargeSmallerImages = false;
121 d->mZoom = 1; 135​ d->mZoom = 1;
122 d->mZoomToFit = true; 136​ d->mZoomToFit = true;
123 d->mZoomToFill = false; 137​ d->mZoomToFill = false;
124 d->mImageOffset = QPointF(0, 0); 138​ d->mImageOffset = QPointF(0, 0);
125 d->mScrollPos = QPointF(0, 0); 139​ d->mScrollPos = QPointF(0, 0);
126 setFocusPolicy(Qt::WheelFocus); 140​ setFocusPolicy(Qt::WheelFocus);
rkflxUnsubmitted
Done

That's probably not needed anymore.

rkflx: That's probably not needed anymore.
127 setFlag(ItemIsSelectable); 141​ setFlag(ItemIsSelectable);
128 setFlag(ItemClipsChildrenToShape); 142​ setFlag(ItemClipsChildrenToShape);
129 setAcceptHoverEvents(true); 143​ setAcceptHoverEvents(true);
130 d->setupZoomCursor(); 144​ d->setupZoomCursor();
131 updateCursor(); 145​ updateCursor();
132} 146​}
133 147​
134AbstractImageView::~AbstractImageView() 148​AbstractImageView::~AbstractImageView()
▲ Show 20 Lines • Show All 118 Lines • ▼ Show 20 Line(s)266 if (on) {
253 setZoom(computeZoomToFill()); 267​ setZoom(computeZoomToFill());
254 } 268​ }
255 // We do not set zoom to 1 if zoomToFit is off, this is up to the code 269​ // We do not set zoom to 1 if zoomToFit is off, this is up to the code
256 // calling us. It may went to zoom to some other level and/or to zoom on 270​ // calling us. It may went to zoom to some other level and/or to zoom on
257 // a particular position 271​ // a particular position
258 zoomToFillChanged(d->mZoomToFill); 272​ zoomToFillChanged(d->mZoomToFill);
259} 273​}
260 274​
275​const QPixmap& AbstractImageView::alphaBackgroundTexture() const
rkflxUnsubmitted
Done

I believe const QPixmap& would avoid a bunch of useless copying.

rkflx: I believe `const QPixmap&` would avoid a bunch of useless copying.
huoniAuthorUnsubmitted
Not Done

It's hard to know when to conform to surrounding code, and when to break that consistency. In hindsight, a QPixmap is going to be more expensive to copy than QRect and the like. so this makes sense.

huoni: It's hard to know when to conform to surrounding code, and when to break that consistency. In…
276​{
277​ return d->mAlphaBackgroundTexture;
278​}
279​
261void AbstractImageView::resizeEvent(QGraphicsSceneResizeEvent* event) 280​void AbstractImageView::resizeEvent(QGraphicsSceneResizeEvent* event)
262{ 281​{
263 QGraphicsWidget::resizeEvent(event); 282​ QGraphicsWidget::resizeEvent(event);
264 if (d->mZoomToFit) { 283​ if (d->mZoomToFit) {
265 // setZoom() calls adjustImageOffset(), but only if the zoom changes. 284​ // setZoom() calls adjustImageOffset(), but only if the zoom changes.
266 // If the view is resized but does not cause a zoom change, we call 285​ // If the view is resized but does not cause a zoom change, we call
267 // adjustImageOffset() ourself. 286​ // adjustImageOffset() ourself.
268 const qreal newZoom = computeZoomToFit(); 287​ const qreal newZoom = computeZoomToFit();
▲ Show 20 Lines • Show All 308 Lines • Show Last 20 Lines