Solved Exact correct size needed
-
@Pl45m4
Looks like I owe you a beer :)/*virtual*/ void MapWindow::resizeEvent(QResizeEvent *event) /*override*/ { const QSize &size(event->size()); // to avoid scrollbars, scene must be resized, and needs to be "a little bit" smaller int width = size.width() - 2; int height = size.height() - 2; scene->setSceneRect(0, 0, width, height); // scale the pixmap to fit, keeping aspect ratio const QPixmap &pixmap(pixmapOriginalMap->scaled(width, height, Qt::KeepAspectRatio)); pixmapMap->setPixmap(pixmap); }
This works much better! I was confusing the pixel issue and the need to resize scene smaller on shrink.
Those
- 2
s are magic :) Without I get scrollbar. There should be a Qt constant for this :) -
Hi,
You have to understand what is the scene and what is the view. In your case, the scene is in pixel units. What the view does is show the scene (or part). The view can show the pixmapItem zoomed but the item size in scene units is still the same.
So, as I posted on the other topic:
- Add the pixmapItem to the scene.
- Call "fitInView(pixmapItem,Qt::KeepAspectRatio)
- Override the "resizeEvent" of your Main Widget or your Main Window and call the "fitInView" there.
- If you want to get the mouse position: create a derived QGraphcisView, and you can do something like this:
void MyGraphicsView::mouseMoveEvent(QMouseEvent *pqEvent) { QGraphicsView::mouseMoveEvent(pqEvent); QPointF qScenePos = mapToScene(pqEvent->pos()); qint32 qX = qScenePos.x(); qint32 qY = qScenePos.y(); emit position(qX,qY); }
This takes the GraphicsView position and maps it into the scene coordinates, in this case your image pixel coordinate.
Note that you have to use the MyGraphicsView class instead of QGraphicsView class. Also I have added the SIGNAL "position" -
@ollarch
Thank you for joining this discussion!QGraphcisView::fitInView()
, yes I remembered that vaguely.I have questions about what you are saying:
-
I looked through articles and they said for best quality scaling of pixmaps I should do it via the
QPixmap::scaled()
I am using [I am struggling to find references for this now...]. Isn't yourfitInView(pixmapItem,Qt::KeepAspectRatio)
way going to "lose me quality"? -
You are doing it by resizing the view. Yet I am not doing that, I am resizing the whole scene to fir the pixmap. As I think advised by @Pl45m4. What I have works (though there are those
- 2
s...). Doesn't it? is it better to do your way via view instead of scene?
-
-
Hi,
The view is the one that takes care of the display part. What it does is show the scene with the indicated transformations. So, let the view do its job.
The view is not resizing, it only paints the scene and does not modify the scene. So, as I told you, the pixmap is still the same, what have changed is where are you looking it.Think on a CAD software. The schematic have to keep the units regardless of the zoom applied or the rotation of the view. You can rotate the view, zoom it, ... but when you ask the view to map the mouse position into scene coordinates, this coordinates will still be the same pixel position.
Start with points 1 and 2. When you have it working, do the point 3. Finally point 4.
When you finish all this steps, call "rotate(90)" on your view and see what happens (what you see and the mouse positions). -
@ollarch
Thank you. I will save what I have now and rewrite your way tomorrow, and report back :) -
@ollarch
Firstly, I do understand what you are saying/your approach. I had been expecting to have to take the current scaling into account and do my own conversion of mouse coords to scene/pixmap coords. I agree that your way means theQGraphicsView
will do this for me, and that makes me a happy bunny :)I followed your instructions above exactly. It almost worked, but there is a little wrinkle!
When the window is first shown the pixmap is "small", does not fit whole area --- even though I had
fitToView()
in the constructor (where I add the pixmap) and aresizeEvent
is generated. Once I then start interactive window resizing it jumps to correct, full size, and maintains that from then on correctly.[Also your way I did not have to put in any "magic"
- 2
on sizes, it never caused scrollbars to appear (even when I still had them allowed), so no need for "small pixel adjustment, which is an added plus point :) ]I tried "everything" (I'm pretty thorough!) but nothing remedied this. So on a hunch I put in a
showEvent()
override to do thefitToView()
there, and sure enough that, and only that, made it also be correct when first shown.The essentials of my now-working code are:
void MapWindow::setupUi() { // as per https://doc.qt.io/qt-5/qgraphicsview.html#fitInView // since we intend to always call `QGraphicsView::fitToView()` in `resizeEvent()` // we explicitly disable scrollbars to *ensure* no infinite recursion view->setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff); view->setVerticalScrollBarPolicy(Qt::ScrollBarAlwaysOff); // create the map as a `QGraphicsPixmapItem` // `pixmapMap` will be fitted tightly in view in `showEvent()`/`resizeEvent()` this->pixmapMap = new QGraphicsPixmapItem(*pixmapOriginalMap); scene->addItem(pixmapMap); } /*virtual*/ void MapWindow::showEvent(QShowEvent *showEvent) /*override*/ { // call the base method QWidget::showEvent(showEvent); view->fitInView(pixmapMap, Qt::KeepAspectRatio); } /*virtual*/ void MapWindow::resizeEvent(QResizeEvent *event) /*override*/ { // call the base method QWidget::resizeEvent(event); view->fitInView(pixmapMap, Qt::KeepAspectRatio); }
[I am unsure about the need to call
QWidget::showEvent
/resizeEvent()
base class, and where, in my overrides. Some examples do, some don't. With/without, or moving/before after thefitInView()
call, all make no difference.]I don't know whether you have any comment/alternative suggestion about this need to do
fitView()
inshowEvent()
, but I couldn't get it to work first time without? -
@JonB said in Exact correct size needed:
When the window is first shown the pixmap is "small", does not fit whole area
This is because when the widget constructor is executed, the widget size is not the "final" widget size. After the constructor is called you call "show" and then it calculates the size of the ui elements.
[I am unsure about the need to call QWidget::showEvent/resizeEvent() base class
You don't need to call it.
Can you check the values on the debugger of "event->s" and "event->olds" ? The first one is the new size and the last is the old size.
A workarround for this could be to use a "QTimer::singleshot(100,this,&YourWindowClass::fitMyPixmapToView)" into constructor.
, but I have not need to use it. -
@ollarch said in Exact correct size needed:
This is because when the widget constructor is executed, the widget size is not the "final" widget size.
Yep, I do realize this :)
After the constructor is called you call "show" and then it calculates the size of the ui elements.
It doesn't matter when I call
show()
, I find I have to do the "fit" inshowEvent()
, i.e. when it's actually shown.Can you check the values on the debugger of "event->s" and "event->olds" ? The first one is the new size and the last is the old size.
I did that right from the start. In the very first
resizeEvent()
the old size is always(-1, -1)
, while the (new) size is always the correct, big one. This is still the case even after I have fitted inshowEvent()
. So, it looks like I get the firstresizeEvent()
before theshowEvent()
, and at that point fitting doesn't work inresizeEvent()
but it does inshowEvent()
.12:33:20: Debugging starts resizeEvent QSize(-1, -1) QSize(1492, 772) showEvent // now start dragging resizeEvent QSize(1492, 772) QSize(1491, 772) resizeEvent QSize(1491, 772) QSize(1490, 772) resizeEvent QSize(1490, 772) QSize(1490, 771) ...
A workarround for this could be to use a "QTimer::singleshot(100,this,&YourWindowClass::fitMyPixmapToView)" into constructor.
Indeed I could have done that. But since it works via
showEvent()
I'm happy.What else can I say? :)
-
Hi, I just found that I use to call "showMaximized" on "main.cpp" instead of "show" to make the application full screen.
I've tested with calling "show" and yes, it does not fit the item to the view the first time.I don't like too much this solution but you can use this
QTimer::singleShot(50, this, SLOT(fit()));
Define "fit" method as slot.
-
@ollarch
Thanks, it sounds like (if I understand correctly) you are saying you find the same issue as me first time? Though I haven't seen this commented on elsewhere.As I said earlier, I have no problem with rectifying via your
QTimer
, but since I have rectified viashowEvent()
(as per my code) why are you not satisfied with that approach. any reason?I have moved on now to mouseworks, and am happy with my coordinates :)
-
As I said earlier, I have no problem with rectifying via your QTimer, but since I have rectified via showEvent() (as per my code) why are you not satisfied with that approach. any reason?
No, I have any problem and I'm satisfied that it works.
I have moved on now to mouseworks, and am happy with my coordinates :)
Now you can try to rotate the view 45 degrees and you will see that when you map the view coordinates to scene coordinates you still have the correct pixel coordinate.
-
@ollarch
Indeed, though rotation is not something I will want to do, but I take your point.Overall I am very happy with the approach you stuck with and made me adopt for my situation. As I said, I thought I was going to have to do my own coordinate transformation arithmetic taking into account my current scale/zoom setting etc. I am much happier that all I have to do is call Qt's already-written
mapToScene()
to do it for me, and everything makes logical, "purist" sense to me.So thank you for sticking to your guns and recommending the approach I was really looking for!
-
Thanks for the clarification. mobdro
-
Overall I am very happy with the approach you stuck with and made me adopt for my situation. As I said, I thought I was going to have to do my own coordinate transformation arithmetic taking into account my current scale/zoom setting etc. I am much happier that all I have to do is call Qt's already-written
mapToScene()
to do it for me, and everything makes logical, "purist" sense to me.I'd rather not reinvent the wheel.
So thank you for sticking to your guns and recommending the approach I was really looking for!
I'm happy. ;)
-
@ollarch
Hi @ollarch , hope you're around and see this question (or anyone else who knows!)....I am moving onto my requirement to draw a shape on the view to follow the mouse. It will be see-through but with a frame, so that user can see where on map the mouse is.
For now I will do that via a
QAbstractGraphicsShapeItem
on theQGrapcicsScene
, because that's the only way I know how to do it; plus it will use scene coordinates, which is easy.However, I am aware this is not "correct". There should be no permanent object on the scene for this. And, for example, although I do not have multiple views onto the scene, if I did it would show up in all views. Which is not correct. This shape is a temporary object which belongs/should be shown only in the view where the mouse is moved, not on the scene shared across views.
The "correct" thing to is to draw this on the
GraphicsView
only (with necessary coordinate calculations, but that's not the point). I came acrossQRubberBand
which looks like the sort of thing which should be used (and I think that can be put on the view only). But unfortunately for me that only allows a line or a rectangle, and I will want a circle or a hexagon for my usage.So.... what is the right way to do this, please? :)
EDIT
I have now come across, say, https://www.qtcentre.org/threads/4479-Adding-Rectangular-overlay-on-graphics-view [from 2006!], which is asking just my kind of question:To simplify my problem, lets assume that we have two different graphics views. Both of these views have the same scene applied to it. When the user clicks on a specific point in the first graphics view I want to draw a rectangle related to the point she clicked, but I want it to only be visible in that graphics view.
I think this ends up saying to use QGraphicsView::drawForeground(QPainter *painter, const QRectF &rect) to achieve this. Is that the route I should be following? [And if it is I shall have further questions! Because the principle there is not good enough as-is for following the mouse and redrawing when it moves.]
UPDATE
Now that I am beginning to understand what facilities the graphics scene/view offer. I am now sailing along, using the foreground layer of the view viaQGraphicsView::drawForeground()
to draw my mouse-follow-shape, callingscene()->invalidate(QRectF(), QGraphicsScene::ForegroundLayer);
to cause it to redraw frommouseMoveEvent()
. I think I have got this right!Which now makes me think: maybe I should be drawing the map on which all this happens via
QGraphicsScene::drawBackground()
instead of in aQGraphicsPixmapItem()
on the scene? For one thing, I believe the scene's background layer is cached for speed by default, which sounds good for an unchanging map background? -
So you want to display a rect only on the first view? You could create two scenes and two views and add only the rect into the first one. If the second view is some kind of "only selected view" you can take the rect position and size to crop the image and set it to the QGraphicsPixmapItem on the second scene.
-
@ollarch said in Exact correct size needed:
Hi @ollarch. Your post has just crossed with the latest UPDATE I have typed into my post above!So you want to display a rect only on the first view? You could create two scenes and two views and add only the rect into the first one.
Nooo, I don't think so! 2 scenes?! Just for a shape on a view?
I'm pretty sure now that I understand a bit more that what I am doing via
QGraphicsView::drawForeground()
is the correct, cheapest way to draw a temporary foreground shape --- which is not an object on the scene --- to follow the mouse.You can see there that I am getting so excited now that I understand layers --- background -> any graphics objects -> foreground --- plus the fact that
QGraphicsView
draws its own layers, QGraphicsView::drawForegroundReimplement this function to provide a custom foreground for this view.
The default implementation fills rect using the view's foregroundBrush. If no such brush is defined (the default), the scene's
drawForeground()
function is called instead.So I now think I ought best move the map pixmap out from being a graphics object and into the scene's background layer instead. I am enjoying this as I begin to understand :)
-
If I understand it. When the mouse enters the view you want to display a rect (or circle) centered to the mouse position?
-
@ollarch
Yes. (Only a touch more complex than that: it has to map to nearest rect/circle/shape where the map is notionally divided into a grid of distinctly-located shapes, rather than continuous, and "snap" to there. But that is a detail.) And this shape must only be drawn on the view where the mouse is being moved; it does not belong to the scene, and if there were multiple views it must not appear in views other than the one with the mouse move. That's why it has to be view-based, not scene-based.But I am already at this point now, I have achieved what I need, using
QGraphicsView::drawForeground()
:) -
Hi,
Do you have a second view, for what?