]> git.sesse.net Git - kdenlive/blobdiff - src/geometryval.cpp
Correctly update monitor when changing a title clip duration
[kdenlive] / src / geometryval.cpp
index 2c40d6e098e651b1b95b8a045ab0a495ecc215fa..a44fe34b5c8dc7280e606d4c9cf5a2aa0ae82cc1 100644 (file)
  *                                                                         *
  ***************************************************************************/
 
+#include "geometryval.h"
+#include "graphicsscenerectmove.h"
+#include "kdenlivesettings.h"
 
+#include <KDebug>
 
 #include <QGraphicsView>
 #include <QVBoxLayout>
-#include <QGraphicsScene>
 #include <QGraphicsRectItem>
-#include <QMouseEvent>
 #include <QMenu>
-
-#include <KDebug>
-
-#include "graphicsscenerectmove.h"
-#include "geometryval.h"
-#include "kdenlivesettings.h"
-
-Geometryval::Geometryval(const MltVideoProfile profile, QWidget* parent): QWidget(parent), m_profile(profile), m_geom(NULL), m_path(NULL), paramRect(NULL), m_fixedMode(false) {
-    ui.setupUi(this);
-    QVBoxLayout* vbox = new QVBoxLayout(ui.widget);
+#include <QInputDialog>
+
+
+Geometryval::Geometryval(const MltVideoProfile profile, QPoint frame_size, QWidget* parent) :
+        QWidget(parent),
+        m_profile(profile),
+        m_paramRect(NULL),
+        m_geom(NULL),
+        m_path(NULL),
+        m_fixedMode(false),
+        m_frameSize(frame_size)
+{
+    setupUi(this);
+    QVBoxLayout* vbox = new QVBoxLayout(widget);
     QGraphicsView *view = new QGraphicsView(this);
     view->setBackgroundBrush(QBrush(Qt::black));
     vbox->addWidget(view);
     vbox->setContentsMargins(0, 0, 0, 0);
 
-    QVBoxLayout* vbox2 = new QVBoxLayout(ui.keyframeWidget);
+    QVBoxLayout* vbox2 = new QVBoxLayout(keyframeWidget);
     m_helper = new KeyframeHelper(this);
     vbox2->addWidget(m_helper);
     vbox2->setContentsMargins(0, 0, 0, 0);
 
     connect(m_helper, SIGNAL(positionChanged(int)), this, SLOT(slotPositionChanged(int)));
-
-    scene = new GraphicsSceneRectMove(this);
-    scene->setTool(TITLE_SELECT);
-    view->setScene(scene);
-    double aspect = (double) profile.sample_aspect_num / profile.sample_aspect_den * profile.width / profile.height;
-    QGraphicsRectItem *m_frameBorder = new QGraphicsRectItem(QRectF(0, 0, profile.width, profile.height));
-    m_frameBorder->setZValue(-1100);
-    m_frameBorder->setBrush(QColor(255, 255, 0, 30));
-    m_frameBorder->setPen(QPen(QBrush(QColor(255, 255, 255, 255)), 1.0, Qt::DashLine));
-    scene->addItem(m_frameBorder);
-
-    ui.buttonNext->setIcon(KIcon("media-skip-forward"));
-    ui.buttonNext->setToolTip(i18n("Go to next keyframe"));
-    ui.buttonPrevious->setIcon(KIcon("media-skip-backward"));
-    ui.buttonPrevious->setToolTip(i18n("Go to previous keyframe"));
-    ui.buttonAdd->setIcon(KIcon("document-new"));
-    ui.buttonAdd->setToolTip(i18n("Add keyframe"));
-    ui.buttonDelete->setIcon(KIcon("edit-delete"));
-    ui.buttonDelete->setToolTip(i18n("Delete keyframe"));
-
-    QMenu *configMenu = new QMenu(i18n("Misc..."), this);
-    ui.buttonMenu->setIcon(KIcon("system-run"));
-    ui.buttonMenu->setMenu(configMenu);
-    ui.buttonMenu->setPopupMode(QToolButton::QToolButton::InstantPopup);
-
+    connect(m_helper, SIGNAL(keyframeMoved(int)), this, SLOT(slotKeyframeMoved(int)));
+    connect(m_helper, SIGNAL(addKeyframe(int)), this, SLOT(slotAddFrame(int)));
+    connect(m_helper, SIGNAL(removeKeyframe(int)), this, SLOT(slotDeleteFrame(int)));
+
+    m_scene = new GraphicsSceneRectMove(this);
+    m_scene->setTool(TITLE_SELECT);
+    view->setScene(m_scene);
+    m_dar = (m_profile.height * m_profile.display_aspect_num / (double) m_profile.display_aspect_den) / (double) m_profile.width;
+
+    m_realWidth = (int)(profile.height * profile.display_aspect_num / (double) profile.display_aspect_den);
+    QGraphicsRectItem *frameBorder = new QGraphicsRectItem(QRectF(0, 0, m_realWidth, profile.height));
+    frameBorder->setZValue(-1100);
+    frameBorder->setBrush(QColor(255, 255, 0, 30));
+    frameBorder->setPen(QPen(QBrush(QColor(255, 255, 255, 255)), 1.0, Qt::DashLine));
+    m_scene->addItem(frameBorder);
+
+    buttonNext->setIcon(KIcon("media-skip-forward"));
+    buttonNext->setToolTip(i18n("Go to next keyframe"));
+    buttonPrevious->setIcon(KIcon("media-skip-backward"));
+    buttonPrevious->setToolTip(i18n("Go to previous keyframe"));
+    buttonAdd->setIcon(KIcon("document-new"));
+    buttonAdd->setToolTip(i18n("Add keyframe"));
+    buttonDelete->setIcon(KIcon("edit-delete"));
+    buttonDelete->setToolTip(i18n("Delete keyframe"));
+
+    m_configMenu = new QMenu(i18n("Misc..."), this);
+    buttonMenu->setIcon(KIcon("system-run"));
+    buttonMenu->setMenu(m_configMenu);
+    buttonMenu->setPopupMode(QToolButton::QToolButton::InstantPopup);
+
+
+    m_editGeom = m_configMenu->addAction(i18n("Edit keyframe"), this, SLOT(slotGeometry()));
 
     m_scaleMenu = new QMenu(i18n("Resize..."), this);
-    configMenu->addMenu(m_scaleMenu);
+    m_configMenu->addMenu(m_scaleMenu);
     m_scaleMenu->addAction(i18n("50%"), this, SLOT(slotResize50()));
     m_scaleMenu->addAction(i18n("100%"), this, SLOT(slotResize100()));
     m_scaleMenu->addAction(i18n("200%"), this, SLOT(slotResize200()));
-
+    m_scaleMenu->addAction(i18n("Original size"), this, SLOT(slotResizeOriginal()));
+    m_scaleMenu->addAction(i18n("Custom"), this, SLOT(slotResizeCustom()));
 
     m_alignMenu = new QMenu(i18n("Align..."), this);
-    configMenu->addMenu(m_alignMenu);
+    m_configMenu->addMenu(m_alignMenu);
     m_alignMenu->addAction(i18n("Center"), this, SLOT(slotAlignCenter()));
     m_alignMenu->addAction(i18n("Hor. Center"), this, SLOT(slotAlignHCenter()));
     m_alignMenu->addAction(i18n("Vert. Center"), this, SLOT(slotAlignVCenter()));
@@ -88,7 +102,7 @@ Geometryval::Geometryval(const MltVideoProfile profile, QWidget* parent): QWidge
     m_alignMenu->addAction(i18n("Bottom"), this, SLOT(slotAlignBottom()));
 
 
-    m_syncAction = configMenu->addAction(i18n("Sync timeline cursor"), this, SLOT(slotSyncCursor()));
+    m_syncAction = m_configMenu->addAction(i18n("Sync timeline cursor"), this, SLOT(slotSyncCursor()));
     m_syncAction->setCheckable(true);
     m_syncAction->setChecked(KdenliveSettings::transitionfollowcursor());
 
@@ -96,138 +110,199 @@ Geometryval::Geometryval(const MltVideoProfile profile, QWidget* parent): QWidge
     //view->fitInView(m_frameBorder, Qt::KeepAspectRatio);
     const double sc = 100.0 / profile.height * 0.8;
     QRectF srect = view->sceneRect();
-    view->setSceneRect(srect.x(), -srect.height() / 3, srect.width(), srect.height() + srect.height() / 3 * 2);
-    scene->setZoom(sc);
-    view->centerOn(m_frameBorder);
-    connect(ui.buttonNext , SIGNAL(clicked()) , this , SLOT(slotNextFrame()));
-    connect(ui.buttonPrevious , SIGNAL(clicked()) , this , SLOT(slotPreviousFrame()));
-    connect(ui.buttonDelete , SIGNAL(clicked()) , this , SLOT(slotDeleteFrame()));
-    connect(ui.buttonAdd , SIGNAL(clicked()) , this , SLOT(slotAddFrame()));
-    connect(scene, SIGNAL(actionFinished()), this, SLOT(slotUpdateTransitionProperties()));
+    view->setSceneRect(srect.x(), -srect.height() / 3 + 10, srect.width(), srect.height() + srect.height() / 3 * 2 - 10);
+    m_scene->setZoom(sc);
+    view->centerOn(frameBorder);
+    connect(buttonNext , SIGNAL(clicked()) , this , SLOT(slotNextFrame()));
+    connect(buttonPrevious , SIGNAL(clicked()) , this , SLOT(slotPreviousFrame()));
+    connect(buttonDelete , SIGNAL(clicked()) , this , SLOT(slotDeleteFrame()));
+    connect(buttonAdd , SIGNAL(clicked()) , this , SLOT(slotAddFrame()));
+    connect(m_scene, SIGNAL(actionFinished()), this, SLOT(slotUpdateTransitionProperties()));
+    connect(m_scene, SIGNAL(doubleClickEvent()), this, SLOT(slotGeometry()));
+
+}
+
+
+Geometryval::~Geometryval()
+{
+    m_scene->disconnect();
+    delete m_scaleMenu;
+    delete m_alignMenu;
+    delete m_editGeom;
+    delete m_syncAction;
+    delete m_configMenu;
+    delete m_paramRect;
+    delete m_path;
+    delete m_helper;
+    delete m_geom;
+    delete m_scene;
+}
+
+
+void Geometryval::slotAlignCenter()
+{
+    int pos = spinPos->value();
+    Mlt::GeometryItem item;
+    int error = m_geom->fetch(&item, pos);
+    if (error || item.key() == false) {
+        // no keyframe under cursor
+        return;
+    }
+    m_paramRect->setPos((m_realWidth - m_paramRect->rect().width()) / 2, (m_profile.height - m_paramRect->rect().height()) / 2);
+    slotUpdateTransitionProperties();
+}
+
+void Geometryval::slotAlignHCenter()
+{
+    int pos = spinPos->value();
+    Mlt::GeometryItem item;
+    int error = m_geom->fetch(&item, pos);
+    if (error || item.key() == false) {
+        // no keyframe under cursor
+        return;
+    }
+    m_paramRect->setPos((m_realWidth - m_paramRect->rect().width()) / 2, m_paramRect->pos().y());
+    slotUpdateTransitionProperties();
 }
 
-void Geometryval::slotAlignCenter() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotAlignVCenter()
+{
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
         // no keyframe under cursor
         return;
     }
-    paramRect->setPos((m_profile.width - paramRect->rect().width()) / 2, (m_profile.height - paramRect->rect().height()) / 2);
+    m_paramRect->setPos(m_paramRect->pos().x(), (m_profile.height - m_paramRect->rect().height()) / 2);
     slotUpdateTransitionProperties();
 }
 
-void Geometryval::slotAlignHCenter() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotAlignTop()
+{
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
         // no keyframe under cursor
         return;
     }
-    paramRect->setPos((m_profile.width - paramRect->rect().width()) / 2, paramRect->pos().y());
+    m_paramRect->setPos(m_paramRect->pos().x(), 0);
     slotUpdateTransitionProperties();
 }
 
-void Geometryval::slotAlignVCenter() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotAlignBottom()
+{
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
         // no keyframe under cursor
         return;
     }
-    paramRect->setPos(paramRect->pos().x(), (m_profile.height - paramRect->rect().height()) / 2);
+    m_paramRect->setPos(m_paramRect->pos().x(), m_profile.height - m_paramRect->rect().height());
     slotUpdateTransitionProperties();
 }
 
-void Geometryval::slotAlignTop() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotAlignLeft()
+{
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
         // no keyframe under cursor
         return;
     }
-    paramRect->setPos(paramRect->pos().x(), 0);
+    m_paramRect->setPos(0, m_paramRect->pos().y());
     slotUpdateTransitionProperties();
 }
 
-void Geometryval::slotAlignBottom() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotAlignRight()
+{
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
         // no keyframe under cursor
         return;
     }
-    paramRect->setPos(paramRect->pos().x(), m_profile.height - paramRect->rect().height());
+    m_paramRect->setPos(m_realWidth - m_paramRect->rect().width(), m_paramRect->pos().y());
     slotUpdateTransitionProperties();
 }
 
-void Geometryval::slotAlignLeft() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotResize50()
+{
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
         // no keyframe under cursor
         return;
     }
-    paramRect->setPos(0, paramRect->pos().y());
+    m_paramRect->setRect(0, 0, m_realWidth / 2, m_profile.height / 2);
     slotUpdateTransitionProperties();
 }
 
-void Geometryval::slotAlignRight() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotResize100()
+{
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
         // no keyframe under cursor
         return;
     }
-    paramRect->setPos(m_profile.width - paramRect->rect().width(), paramRect->pos().y());
+    m_paramRect->setRect(0, 0, m_realWidth, m_profile.height);
     slotUpdateTransitionProperties();
 }
 
-void Geometryval::slotResize50() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotResize200()
+{
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
         // no keyframe under cursor
         return;
     }
-    paramRect->setRect(0, 0, m_profile.width / 2, m_profile.height / 2);
+    m_paramRect->setRect(0, 0, m_realWidth * 2, m_profile.height * 2);
     slotUpdateTransitionProperties();
 }
 
-void Geometryval::slotResize100() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotResizeOriginal()
+{
+    if (m_frameSize.isNull()) slotResize100();
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
         // no keyframe under cursor
         return;
     }
-    paramRect->setRect(0, 0, m_profile.width, m_profile.height);
+    m_paramRect->setRect(0, 0, m_frameSize.x(), m_frameSize.y());
     slotUpdateTransitionProperties();
 }
 
-void Geometryval::slotResize200() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotResizeCustom()
+{
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
         // no keyframe under cursor
         return;
     }
-    paramRect->setRect(0, 0, m_profile.width * 2, m_profile.height * 2);
+    int scale = m_paramRect->rect().width() * 100 / m_realWidth;
+    bool ok;
+    scale =  QInputDialog::getInteger(this, i18n("Resize..."), i18n("Scale"), scale, 1, 2147483647, 10, &ok);
+    if (!ok) return;
+    m_paramRect->setRect(0, 0, m_realWidth * scale / 100, m_profile.height * scale / 100);
     slotUpdateTransitionProperties();
 }
 
-void Geometryval::slotTransparencyChanged(int transp) {
-    int pos = ui.spinPos->value();
+void Geometryval::slotTransparencyChanged(int transp)
+{
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
@@ -235,124 +310,140 @@ void Geometryval::slotTransparencyChanged(int transp) {
         return;
     }
     item.mix(transp);
-    paramRect->setBrush(QColor(255, 0, 0, transp));
+    m_paramRect->setBrush(QColor(255, 0, 0, transp));
     m_geom->insert(item);
     emit parameterChanged();
 }
 
-void Geometryval::slotSyncCursor() {
+void Geometryval::slotSyncCursor()
+{
     KdenliveSettings::setTransitionfollowcursor(m_syncAction->isChecked());
 }
 
-void Geometryval::slotPositionChanged(int pos, bool seek) {
+void Geometryval::slotPositionChanged(int pos, bool seek)
+{
     if (seek && KdenliveSettings::transitionfollowcursor()) emit seekToPos(pos);
-    ui.spinPos->setValue(pos);
+    spinPos->setValue(pos);
     m_helper->setValue(pos);
     Mlt::GeometryItem item;
     int error = m_geom->fetch(&item, pos);
     if (error || item.key() == false) {
         // no keyframe under cursor, adjust buttons
-        ui.buttonAdd->setEnabled(true);
-        ui.buttonDelete->setEnabled(false);
-        ui.widget->setEnabled(false);
-        ui.spinTransp->setEnabled(false);
+        buttonAdd->setEnabled(true);
+        buttonDelete->setEnabled(false);
+        widget->setEnabled(false);
+        spinTransp->setEnabled(false);
         m_scaleMenu->setEnabled(false);
         m_alignMenu->setEnabled(false);
+        m_editGeom->setEnabled(false);
     } else {
-        ui.buttonAdd->setEnabled(false);
-        ui.buttonDelete->setEnabled(true);
-        ui.widget->setEnabled(true);
-        ui.spinTransp->setEnabled(true);
+        buttonAdd->setEnabled(false);
+        buttonDelete->setEnabled(true);
+        widget->setEnabled(true);
+        spinTransp->setEnabled(true);
         m_scaleMenu->setEnabled(true);
         m_alignMenu->setEnabled(true);
+        m_editGeom->setEnabled(true);
     }
-    paramRect->setPos(item.x(), item.y());
-    paramRect->setRect(0, 0, item.w(), item.h());
-    ui.spinTransp->setValue(item.mix());
-    paramRect->setBrush(QColor(255, 0, 0, item.mix()));
+
+    m_paramRect->setPos(item.x() * m_dar, item.y());
+    m_paramRect->setRect(0, 0, item.w() * m_dar, item.h());
+    spinTransp->setValue(item.mix());
+    m_paramRect->setBrush(QColor(255, 0, 0, item.mix()));
 }
 
-void Geometryval::slotDeleteFrame() {
+void Geometryval::slotDeleteFrame(int pos)
+{
     // check there is more than one keyframe
     Mlt::GeometryItem item;
-    const int pos = ui.spinPos->value();
+    if (pos == -1) pos = spinPos->value();
     int error = m_geom->next_key(&item, pos + 1);
     if (error) {
         error = m_geom->prev_key(&item, pos - 1);
         if (error || item.frame() == pos) return;
     }
 
-    m_geom->remove(ui.spinPos->value());
-    ui.buttonAdd->setEnabled(true);
-    ui.buttonDelete->setEnabled(false);
-    ui.widget->setEnabled(false);
-    ui.spinTransp->setEnabled(false);
+    m_geom->remove(spinPos->value());
+    buttonAdd->setEnabled(true);
+    buttonDelete->setEnabled(false);
+    widget->setEnabled(false);
+    spinTransp->setEnabled(false);
     m_scaleMenu->setEnabled(false);
     m_alignMenu->setEnabled(false);
+    m_editGeom->setEnabled(false);
     m_helper->update();
+    slotPositionChanged(pos, false);
+    updateTransitionPath();
+    emit parameterChanged();
 }
 
-void Geometryval::slotAddFrame() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotAddFrame(int pos)
+{
+    if (pos == -1) pos = spinPos->value();
     Mlt::GeometryItem item;
     item.frame(pos);
-    item.x(paramRect->pos().x());
-    item.y(paramRect->pos().y());
-    item.w(paramRect->rect().width());
-    item.h(paramRect->rect().height());
-    item.mix(ui.spinTransp->value());
+    QRectF r = m_paramRect->sceneBoundingRect().normalized();
+    item.x(r.left() / m_dar);
+    item.y(r.top());
+    item.w(r.width() / m_dar);
+    item.h(r.height());
+    item.mix(spinTransp->value());
     m_geom->insert(item);
-    ui.buttonAdd->setEnabled(false);
-    ui.buttonDelete->setEnabled(true);
-    ui.widget->setEnabled(true);
-    ui.spinTransp->setEnabled(true);
+    buttonAdd->setEnabled(false);
+    buttonDelete->setEnabled(true);
+    widget->setEnabled(true);
+    spinTransp->setEnabled(true);
     m_scaleMenu->setEnabled(true);
     m_alignMenu->setEnabled(true);
+    m_editGeom->setEnabled(true);
     m_helper->update();
+    emit parameterChanged();
 }
 
-void Geometryval::slotNextFrame() {
+void Geometryval::slotNextFrame()
+{
     Mlt::GeometryItem item;
     int error = m_geom->next_key(&item, m_helper->value() + 1);
     kDebug() << "// SEEK TO NEXT KFR: " << error;
     if (error) {
         // Go to end
-        ui.spinPos->setValue(ui.spinPos->maximum());
+        spinPos->setValue(spinPos->maximum());
         return;
     }
     int pos = item.frame();
-    ui.spinPos->setValue(pos);
+    spinPos->setValue(pos);
 }
 
-void Geometryval::slotPreviousFrame() {
+void Geometryval::slotPreviousFrame()
+{
     Mlt::GeometryItem item;
     int error = m_geom->prev_key(&item, m_helper->value() - 1);
     kDebug() << "// SEEK TO NEXT KFR: " << error;
     if (error) return;
     int pos = item.frame();
-    ui.spinPos->setValue(pos);
+    spinPos->setValue(pos);
 }
 
 
-QDomElement Geometryval::getParamDesc() {
-    param.setAttribute("value", m_geom->serialise());
-    kDebug() << " / / UPDATING TRANSITION VALUE: " << param.attribute("value");
-    return param;
+QString Geometryval::getValue() const
+{
+    return m_geom->serialise();
 }
 
-void Geometryval::setupParam(const QDomElement& par, int minFrame, int maxFrame) {
-    param = par;
+void Geometryval::setupParam(const QDomElement par, int minFrame, int maxFrame)
+{
     QString val = par.attribute("value");
     if (par.attribute("fixed") == "1") {
         m_fixedMode = true;
-        ui.buttonPrevious->setHidden(true);
-        ui.buttonNext->setHidden(true);
-        ui.buttonDelete->setHidden(true);
-        ui.buttonAdd->setHidden(true);
-        ui.spinTransp->setMaximum(500);
-        ui.label_pos->setHidden(true);
+        buttonPrevious->setHidden(true);
+        buttonNext->setHidden(true);
+        buttonDelete->setHidden(true);
+        buttonAdd->setHidden(true);
+        spinTransp->setMaximum(500);
+        label_pos->setHidden(true);
         m_helper->setHidden(true);
-        ui.spinPos->setHidden(true);
+        spinPos->setHidden(true);
+
     }
     char *tmp = (char *) qstrdup(val.toUtf8().data());
     if (m_geom) m_geom->parse(tmp, maxFrame - minFrame, m_profile.width, m_profile.height);
@@ -367,32 +458,34 @@ void Geometryval::setupParam(const QDomElement& par, int minFrame, int maxFrame)
         /*QDomDocument doc;
         doc.appendChild(doc.importNode(par, true));
         kDebug() << "IMPORTED TRANS: " << doc.toString();*/
-        ui.spinPos->setMaximum(maxFrame - minFrame - 1);
+        spinPos->setMaximum(maxFrame - minFrame - 1);
         if (m_path == NULL) {
             m_path = new QGraphicsPathItem();
             m_path->setPen(QPen(Qt::red));
-            scene->addItem(m_path);
+            m_scene->addItem(m_path);
         }
         updateTransitionPath();
     }
     Mlt::GeometryItem item;
 
     m_geom->fetch(&item, 0);
-    if (paramRect) delete paramRect;
-    paramRect = new QGraphicsRectItem(QRectF(0, 0, item.w(), item.h()));
-    paramRect->setPos(item.x(), item.y());
-    paramRect->setZValue(0);
-
-    paramRect->setPen(QPen(QBrush(QColor(255, 0, 0, 255)), 1.0));
-    scene->addItem(paramRect);
+    delete m_paramRect;
+    m_paramRect = new QGraphicsRectItem(QRectF(0, 0, item.w() * m_dar, item.h()));
+    m_paramRect->setPos(item.x() * m_dar, item.y());
+    m_paramRect->setZValue(0);
+    m_paramRect->setFlags(QGraphicsItem::ItemIsMovable | QGraphicsItem::ItemIsSelectable);
+
+    m_paramRect->setPen(QPen(QBrush(QColor(255, 0, 0, 255)), 1.0));
+    m_scene->addItem(m_paramRect);
     slotPositionChanged(0, false);
     if (!m_fixedMode) {
-        connect(ui.spinPos, SIGNAL(valueChanged(int)), this , SLOT(slotPositionChanged(int)));
+        connect(spinPos, SIGNAL(valueChanged(int)), this , SLOT(slotPositionChanged(int)));
     }
-    connect(ui.spinTransp, SIGNAL(valueChanged(int)), this , SLOT(slotTransparencyChanged(int)));
+    connect(spinTransp, SIGNAL(valueChanged(int)), this , SLOT(slotTransparencyChanged(int)));
 }
 
-void Geometryval::updateTransitionPath() {
+void Geometryval::updateTransitionPath()
+{
     if (m_fixedMode) return;
     Mlt::GeometryItem item;
     int pos = 0;
@@ -402,9 +495,9 @@ void Geometryval::updateTransitionPath() {
         if (m_geom->next_key(&item, pos) == 1) break;
         pos = item.frame();
         if (counter == 0) {
-            path.moveTo(item.x() + item.w() / 2, item.y() + item.h() / 2);
+            path.moveTo(item.x() * m_dar + item.w() * m_dar / 2, item.y() + item.h() / 2);
         } else {
-            path.lineTo(item.x() + item.w() / 2, item.y() + item.h() / 2);
+            path.lineTo(item.x() * m_dar + item.w() * m_dar / 2, item.y() + item.h() / 2);
         }
         counter++;
         pos++;
@@ -412,19 +505,84 @@ void Geometryval::updateTransitionPath() {
     m_path->setPath(path);
 }
 
-void Geometryval::slotUpdateTransitionProperties() {
-    int pos = ui.spinPos->value();
+void Geometryval::slotUpdateTransitionProperties()
+{
+    int pos = spinPos->value();
     Mlt::GeometryItem item;
     int error = m_geom->next_key(&item, pos);
     if (error || item.frame() != pos) {
         // no keyframe under cursor
         return;
     }
-    item.x(paramRect->pos().x());
-    item.y(paramRect->pos().y());
-    item.w(paramRect->rect().width());
-    item.h(paramRect->rect().height());
+    QRectF r = m_paramRect->sceneBoundingRect().normalized();
+    item.x(r.left() / m_dar);
+    item.y(r.top());
+    item.w(r.width() / m_dar);
+    item.h(r.height());
     m_geom->insert(item);
     updateTransitionPath();
     emit parameterChanged();
 }
+
+void Geometryval::slotGeometry()
+{
+    int pos = spinPos->value();
+    Mlt::GeometryItem item;
+    int error = m_geom->fetch(&item, pos);
+    if (error || item.key() == false) {
+        // no keyframe under cursor
+        return;
+    }
+    QRectF r = m_paramRect->sceneBoundingRect().normalized();
+
+    QDialog d(this);
+    m_view.setupUi(&d);
+    d.setWindowTitle(i18n("Frame Geometry"));
+    m_view.value_x->setMaximum(10000);
+    m_view.value_x->setMinimum(-10000);
+    m_view.value_y->setMaximum(10000);
+    m_view.value_y->setMinimum(-10000);
+    m_view.value_width->setMaximum(500000);
+    m_view.value_width->setMinimum(1);
+    m_view.value_height->setMaximum(500000);
+    m_view.value_height->setMinimum(1);
+
+    m_view.value_x->setValue(m_paramRect->pos().x());
+    m_view.value_y->setValue(m_paramRect->pos().y());
+    m_view.value_width->setValue(r.width());
+    m_view.value_height->setValue(r.height());
+    connect(m_view.button_reset , SIGNAL(clicked()) , this , SLOT(slotResetPosition()));
+
+    if (d.exec() == QDialog::Accepted) {
+        m_paramRect->setPos(m_view.value_x->value(), m_view.value_y->value());
+        m_paramRect->setRect(0, 0, m_view.value_width->value(), m_view.value_height->value());
+        slotUpdateTransitionProperties();
+    }
+}
+
+void Geometryval::slotResetPosition()
+{
+    m_view.value_x->setValue(0);
+    m_view.value_y->setValue(0);
+
+    if (m_frameSize.isNull()) {
+        m_view.value_width->setValue(m_realWidth);
+        m_view.value_height->setValue(m_profile.height);
+    } else {
+        m_view.value_width->setValue(m_frameSize.x());
+        m_view.value_height->setValue(m_frameSize.y());
+    }
+}
+
+void Geometryval::setFrameSize(QPoint p)
+{
+    m_frameSize = p;
+}
+
+
+void Geometryval::slotKeyframeMoved(int pos)
+{
+    slotPositionChanged(pos);
+    slotUpdateTransitionProperties();
+}
+