diff --git a/src/domain/UBGraphicsItemDelegate.cpp b/src/domain/UBGraphicsItemDelegate.cpp index e279cfbe..fe9abe65 100644 --- a/src/domain/UBGraphicsItemDelegate.cpp +++ b/src/domain/UBGraphicsItemDelegate.cpp @@ -418,7 +418,6 @@ bool UBGraphicsItemDelegate::isLocked() void UBGraphicsItemDelegate::duplicate() { - // TODO UB 4.x .. rewrite .. .this is absurde ... we know what we are duplicating UBApplication::boardController->duplicateItem(dynamic_cast(delegated())); } @@ -1190,8 +1189,6 @@ void DelegateMediaControl::positionHandles() mLCDTimerArea.setHeight(parentItem()->boundingRect().height()); lcdTimer->setRect(mLCDTimerArea); lcdTimer->setPos(mSeecArea.width()-mLCDTimerArea.width(),0); - //lcdTimer->setRect(mLCDTimerArea); - //lcdTimer->setPos(mSeecArea.width()-mLCDTimerArea.width(),0); mSeecArea.setWidth(rect().width()-mLCDTimerArea.width()); @@ -1200,8 +1197,6 @@ void DelegateMediaControl::positionHandles() setRect(selfRect); lcdTimer->setPos(rect().width() - mLCDTimerArea.width(), 0); - //lcdTimer->setPos(rect().width() - mLCDTimerArea.width(), 0); - } void DelegateMediaControl::update() @@ -1209,7 +1204,6 @@ void DelegateMediaControl::update() QTime t; t = t.addMSecs(mCurrentTimeInMs < 0 ? 0 : mCurrentTimeInMs); lcdTimer->display(t.toString("m:ss")); - //lcdTimer->display(t.toString("m:ss")); QGraphicsRectItem::update(); } @@ -1262,7 +1256,6 @@ void DelegateMediaControl::seekToMousePos(QPointF mousePos) minX = frameWidth; length = mSeecArea.width() - lcdTimer->rect().width(); - length = mSeecArea.width() /*- lcdTimer->rect().width()*/; qreal mouseX = mousePos.x(); if (mouseX >= (mSeecArea.width() - mSeecArea.height()/2)) diff --git a/src/domain/UBGraphicsItemDelegate.h b/src/domain/UBGraphicsItemDelegate.h index f135f66b..811b26a2 100644 --- a/src/domain/UBGraphicsItemDelegate.h +++ b/src/domain/UBGraphicsItemDelegate.h @@ -284,7 +284,7 @@ class UBGraphicsItemDelegate : public QObject qreal mAntiScaleRatio; QList mButtons; - + QList mToolBarButtons; UBGraphicsToolBarItem* mToolBarItem; protected slots: diff --git a/src/domain/UBGraphicsMediaItemDelegate.cpp b/src/domain/UBGraphicsMediaItemDelegate.cpp index 8411be34..40c93919 100644 --- a/src/domain/UBGraphicsMediaItemDelegate.cpp +++ b/src/domain/UBGraphicsMediaItemDelegate.cpp @@ -94,7 +94,7 @@ void UBGraphicsMediaItemDelegate::buildButtons() connect(mMuteButton, SIGNAL(clicked(bool)), delegated(), SLOT(toggleMute())); connect(mMuteButton, SIGNAL(clicked(bool)), this, SLOT(toggleMute())); // for changing button image - mButtons << mPlayPauseButton << mStopButton << mMuteButton; + mToolBarButtons << mPlayPauseButton << mStopButton << mMuteButton; mToolBarItem->setItemsOnToolBar(QList() << mPlayPauseButton << mStopButton << mMediaControl << mMuteButton); mToolBarItem->setVisibleOnBoard(true); @@ -156,13 +156,10 @@ void UBGraphicsMediaItemDelegate::positionHandles() int toolBarMinimumWidth = 0; int mediaItemWidth = mToolBarItem->boundingRect().width(); - foreach (DelegateButton* button, mButtons) + foreach (DelegateButton* button, mToolBarButtons) { - if (button->getSection() == Qt::TitleBarArea) - { - mediaItemWidth -= button->boundingRect().width() + mToolBarItem->getElementsPadding(); - toolBarMinimumWidth += button->boundingRect().width() + mToolBarItem->getElementsPadding(); - } + mediaItemWidth -= button->boundingRect().width() + mToolBarItem->getElementsPadding(); + toolBarMinimumWidth += button->boundingRect().width() + mToolBarItem->getElementsPadding(); } toolBarMinimumWidth += mToolBarItem->boundingRect().height(); @@ -196,8 +193,6 @@ void UBGraphicsMediaItemDelegate::remove(bool canUndo) QGraphicsScene* scene = mDelegated->scene(); - scene->removeItem(mMediaControl); - UBGraphicsItemDelegate::remove(canUndo); }