kotschak/qelectrotech-source-mirror
0
0
Fork 0
mirror of https://github.com/qelectrotech/qelectrotech-source-mirror.git synced 2026-10-09 21:54:13 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #1413 from bhangart/fix/image-rotation-saved

Browse Source 
Save the rotation of a picture turned with Space
This commit is contained in:
ispyisail
2026-10-09 22:44:40 +13:00
committed by GitHub
parent af978ab907 136eb08a1b
commit b769fa1213
2 changed files with 20 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
sources/undocommand/rotateselectioncommand.cpp
+19 -6
View File
@@ -109,9 +109,17 @@ m_diagram(diagram)
}
break;
case DiagramImageItem::Type:
m_undo << new QPropertyUndoCommand(item->toGraphicsObject(), "rotation", QVariant(item->rotation()), QVariant(item->rotation()+angle), this);
{
//A picture keeps its rotation in its own transform
//("rotationAngle", around its pivot), which is what
//toXml() saves. QGraphicsItem's plain "rotation" is
//not saved, so a picture turned with it came back
//unrotated once the project was reopened.
auto *image = static_cast<DiagramImageItem *>(item);
m_undo << new QPropertyUndoCommand(image, "rotationAngle", QVariant(image->rotationAngle()), QVariant(image->rotationAngle()+angle), this);
if (rotate_as_group)
addGroupPositionUndo(item, pivot, angle);
addGroupPositionUndo(item, pivot, angle, image->mapToScene(image->pivot()) - image->pos());
}
break;
default:
break;
@@ -135,11 +143,16 @@ m_diagram(diagram)
@param item : item to reposition, its own rotation undo already queued
@param pivot : shared pivot point, in scene coordinates
@param angle : rotation angle in degrees
@param anchor : the point of @a item, relative to its pos(), that
orbits @a pivot -- null for items that turn around pos() itself, the
picture's own pivot for a DiagramImageItem, which turns around that.
Only a null anchor is snapped to the grid: the centre of a picture is
rarely on it.
*/
void RotateSelectionCommand::addGroupPositionUndo(QGraphicsItem *item, const QPointF &pivot, qreal angle)
void RotateSelectionCommand::addGroupPositionUndo(QGraphicsItem *item, const QPointF &pivot, qreal angle, const QPointF &anchor)
{
const QPointF old_pos = item->pos();
const QPointF delta = old_pos - pivot;
const QPointF delta = old_pos + anchor - pivot;
/* Exact arithmetic for the right angles instead of qCos()/qSin().
* The rotate actions only ever pass multiples of 90 degrees, and
@@ -168,8 +181,8 @@ void RotateSelectionCommand::addGroupPositionUndo(QGraphicsItem *item, const QPo
delta.x() * qSin(radians) + delta.y() * qCos(radians));
}
QPointF new_pos = pivot + offset;
if (exact_quadrant)
QPointF new_pos = pivot + offset - anchor;
if (exact_quadrant && anchor.isNull())
{
/* Swapping X/Y deltas for a 90/270 turn only stays on the
* user's configured grid if xGrid == yGrid. With an
sources/undocommand/rotateselectioncommand.h
+1 -1
View File
@@ -42,7 +42,7 @@ class RotateSelectionCommand : public QUndoCommand
bool isValid();
private:
void addGroupPositionUndo(QGraphicsItem *item, const QPointF &pivot, qreal angle);
void addGroupPositionUndo(QGraphicsItem *item, const QPointF &pivot, qreal angle, const QPointF &anchor = QPointF());
Diagram *m_diagram =nullptr;