How to implement a shape resizer with QGraphicsPolygonItem?

[xd123]

Hi,
Yes that sounds like a solution that would solve my issue, is there any example available online that I could bounce from? I looked through QtTransform library but I can't see a way I could implement it with a mouse event when the user would try to drag for example a triangle shape like this:

QPolygonF Triangle;
    Triangle.append(QPointF(-200.,0));
    Triangle.append(QPointF(0.,-200));
    Triangle.append(QPointF(200.,0));
 
 
    QModelIndex index = ui->listView->currentIndex(); //annotating the shape with a selected name from a text file
    QString itemText = index.data(Qt::DisplayRole).toString();
 
    QGraphicsPolygonItem* triangleItem = new QGraphicsPolygonItem(Triangle);
    QColor brush_color(Qt::green); //fill color
    brush_color.setAlpha(50); // alpha index makes brush color more opaque
    QPen blackpen(Qt::black); //border color
    blackpen.setWidth(2); // border width
    triangleItem->setBrush(brush_color); 
    triangleItem->setPen(blackpen);
    triangleItem->setFlag(QGraphicsItem::ItemIsMovable); // making the object draggable across graphics view
    scene->addItem(triangleItem);
 
    QGraphicsTextItem *RectText = new QGraphicsTextItem(itemText, triangleItem);

QPolygonF Triangle;
    Triangle.append(QPointF(-200.,0));
    Triangle.append(QPointF(0.,-200));
    Triangle.append(QPointF(200.,0));


    QModelIndex index = ui->listView->currentIndex(); //annotating the shape with a selected name from a text file
    QString itemText = index.data(Qt::DisplayRole).toString();

    QGraphicsPolygonItem* triangleItem = new QGraphicsPolygonItem(Triangle);
    QColor brush_color(Qt::green); //fill color
    brush_color.setAlpha(50); // alpha index makes brush color more opaque
    QPen blackpen(Qt::black); //border color
    blackpen.setWidth(2); // border width
    triangleItem->setBrush(brush_color); 
    triangleItem->setPen(blackpen);
    triangleItem->setFlag(QGraphicsItem::ItemIsMovable); // making the object draggable across graphics view
    scene->addItem(triangleItem);

    QGraphicsTextItem *RectText = new QGraphicsTextItem(itemText, triangleItem);

To copy to clipboard, switch view to plain text mode 

All my polygon items are done in this way, just different shapes.
Thanks for your help.

[d_stranz]

I don't see what this code has to do with your PolygonResizer code you posted earlier. As I said, in the operator() of that class, you have the new rect and you can retrieve the current boundingRect() of the polygon. The ratio of the two heights gives you the y scale, the ratio of the two widths gives the x scale. You retrieve the QPolygon from the QGraphicsPolygonItem, construct a QTransform from the two scales, and apply that to the polygon to get a new scaled polygon. Set that polygon on the QGraphicsPolygonItem instance.

Wikipedia will tell you how to construct the transform matrix using homogeneous coordinates.

The only trick is that the transform scales about the origin (0, 0) point. If this origin is not at the center of your polygon, scaling may also result in translation in one direction and/or the other. So to fix this, you translate to (0,0), scale, then translate back to the original point.

[xd123]

Well my polygon resizer would work with this triangle I just posted, and similar shapes with 5 and more points. But I can't figure the transformation out from what you are saying, if you could maybe please post a example or code snippet that might help me get on the right track. I watched a few videos from the official Qt youtube channel for transformation and translation of graphics items but can't really see the correlation with my issue, besides using bounding rect.

[d_stranz]

Maybe I don't understand your issue. I thought you were trying to implement resizing of graphics items using resize handles on the bounding rectangles of the items.

What you implemented in your initial post works fine for rectangles and ellipses because they are defined by their bounding rectangles. Polygons and other items are defined by an arbitrary closed set of points connected by lines, but they do have a bounding rectangle, the smallest rectangle that contains all of the points. QPolygon (on which QGraphicsPolygonItem is based) has a constructor that takes a QRect argument, but that creates a rectangle polygon. That is the problem you ran into when you tried to implement the same code for resizing polygons that you used for the rectangle and ellipse.

The Wikipedia article I pointed you to shows how to construct a scale transformation using homogenous coordinates, which is what QTransform uses. Your resize code needs to look something like this (and of course you will add checks to avoid divide by zero):

float sx = float( rect.width() ) / float( boundingRect.width() );
float sy = float( rect.height() ) / float( boundingRect.height() );
 
QTransform transform;
transform.scale( sx, sy );
 
QPolygonF  oldPolygon = item->polygon();
QPolygonF newPolygon = oldPolygon * transform;
 
item->setPolygon( newPolygon );

float sx = float( rect.width() ) / float( boundingRect.width() );
float sy = float( rect.height() ) / float( boundingRect.height() );

QTransform transform;
transform.scale( sx, sy );

QPolygonF  oldPolygon = item->polygon();
QPolygonF newPolygon = oldPolygon * transform;

item->setPolygon( newPolygon );

To copy to clipboard, switch view to plain text mode 

This code will change the aspect ratio. If you want to keep the same aspect ratio, then you will need to change one of the scale factors so that the new width and height preserve the same ratio as the old width and height.

But like I said earlier, the scale transformation scales the object around its (0,0) point. If the polygon's origin (0,0) is not at the center of the polygon, then scaling will probably also move it. If that happens to you, you will have to scale in three steps: translate your polygon to (0,0), scale, translate back. But by default, QGraphicsItem coordinates have the origin at their center (except for QGraphicsTextItem, QGraphicsPixmapItem, and maybe a few others that have their origin at topLeft).

[xd123]

Thank you so much. I now completely see what you meant, I'm sorry for not understanding it the first time :D working with Qt for just about 4 months now. I implemented your idea and it works just as I wanted it to. Really appreciate your help and willingness to explain your solution.

If anybody stumbles on this issue too here is the working polygon resizer class:

class PolygonResizer : public SizeGripItem::Resizer // resize class for polygon
        {
            public:
                virtual void operator()(QGraphicsItem* item, const QRectF& rect)
                {
                    QGraphicsPolygonItem* polygonItem =
                        dynamic_cast<QGraphicsPolygonItem*>(item);
 
                    QRectF itemBoundingRect = item->boundingRect();
                    float sx = float( rect.width() ) / float( itemBoundingRect.width() );
                    float sy = float( rect.height() ) / float( itemBoundingRect.height() );
 
                    QTransform transform;
                    transform.scale( sx, sy );
 
                    QPolygonF  oldPolygon = polygonItem->polygon();
                    QPolygonF newPolygon = oldPolygon * transform;
 
                    if (polygonItem)
                    {
                        polygonItem->setPolygon( newPolygon );
                    }
                }
        };

class PolygonResizer : public SizeGripItem::Resizer // resize class for polygon
        {
            public:
                virtual void operator()(QGraphicsItem* item, const QRectF& rect)
                {
                    QGraphicsPolygonItem* polygonItem =
                        dynamic_cast<QGraphicsPolygonItem*>(item);

                    QRectF itemBoundingRect = item->boundingRect();
                    float sx = float( rect.width() ) / float( itemBoundingRect.width() );
                    float sy = float( rect.height() ) / float( itemBoundingRect.height() );

                    QTransform transform;
                    transform.scale( sx, sy );

                    QPolygonF  oldPolygon = polygonItem->polygon();
                    QPolygonF newPolygon = oldPolygon * transform;

                    if (polygonItem)
                    {
                        polygonItem->setPolygon( newPolygon );
                    }
                }
        };

To copy to clipboard, switch view to plain text mode 

[d_stranz]

Happy to help. A couple of suggestions:

- Since itemBoundingRect and rect are already QRectF (which is floating point based), there is no need to cast width() and height() to float. They already are. I was assuming they were QRect, which is integer based and thus needed casting to avoid rounding errors on division. (if either height or width of the rect is less than the height or width of the bounding rect, integer division would always yield zero as the scale factor).

- You should move the check on polygonItem validity to right after the dynamic_cast. If it is null, then retrieving the oldPolygon a few lines later will blow up.

- You should also use qgraphicsitem_cast<> instead of dynamic_cast<> in all of your resizing methods.