... | ... | @@ -9,24 +9,32 @@ Add target in CMake to produce a run configuration that starts QtCreator with th |
|
|
This list is aspects of the Visual Editor in SpaceCreator that needs restucturing.
|
|
|
|
|
|
|
|
|
`VERectGraphicsItem` has methods `setGeometry` and `setRect`. SetGeometry calls setBoundingRect which should not exits. The whole method needs cleanup.
|
|
|
#### `VERectGraphicsItem` has methods `setGeometry` and `setRect`.
|
|
|
SetGeometry calls setBoundingRect which should not exits. The whole method needs cleanup.
|
|
|
|
|
|
`VEConnectionEndPointGraphicsItem::adjustItem()` needs more descriptive name
|
|
|
#### `VEConnectionEndPointGraphicsItem::adjustItem()` needs more descriptive name
|
|
|
|
|
|
`VEConnectionEndPointGraphicsItem::updateTextPosition()` does not update, it resets.
|
|
|
#### `VEConnectionEndPointGraphicsItem::updateTextPosition()` does not update, it resets.
|
|
|
|
|
|
`VEInteractiveObject::updateTextPosition()` does not update text position. It sets explicit size to the boundingbox of the parent... which makes no sense.
|
|
|
#### `VEInteractiveObject::updateTextPosition()` does not update text position.
|
|
|
It sets explicit size to the boundingbox of the parent... which makes no sense.
|
|
|
|
|
|
What is the meanining of `graphicsviewutils::isBounded`?
|
|
|
#### What is the meanining of `graphicsviewutils::isBounded`?
|
|
|
|
|
|
Does `GripPointsHandler` need to inherit `QGraphicsObject`?
|
|
|
#### Does `GripPointsHandler` need to inherit `QGraphicsObject`?
|
|
|
|
|
|
`TextItem::setExplicitSize` calls `setTextMargin(textMargin()); // ??`
|
|
|
#### `TextItem::setExplicitSize` calls `setTextMargin(textMargin()); // ??`
|
|
|
|
|
|
`EndToEndView::refreshView()` is loooong
|
|
|
#### `EndToEndView::refreshView()` is loooong
|
|
|
|
|
|
`VERectGraphicsItem::onGeometryChanged` add comments or extract method.
|
|
|
#### `VERectGraphicsItem::onGeometryChanged` add comments or extract method.
|
|
|
|
|
|
`VEInteractiveObject::itemLevel` is weird. Why does that need to be a pure virtual method and not just a getter that uses the gi::itemLevel
|
|
|
#### `VEInteractiveObject::itemLevel` is weird.
|
|
|
Why does that need to be a pure virtual method and not just a getter that uses the gi::itemLevel
|
|
|
|
|
|
`GripPoint::updateLayout()` does not update a layout but sets the cursor-icon and the position of the grippoint. The position changes often, the cursor-icon only once. split the method. |
|
|
\ No newline at end of file |
|
|
#### `GripPoint::updateLayout()`
|
|
|
It does not update a layout but sets the cursor-icon and the position of the grippoint. The position changes often, the cursor-icon only once. split the method.
|
|
|
|
|
|
#### Item position aligment
|
|
|
There should be helpers, to align functions have function at the same left/top/right/bottom position.
|
|
|
That cluld be lines indicatingthe alignment. And maybe snap to the lines. |
|
|
\ No newline at end of file |