]> git.sesse.net Git - kdenlive/blobdiff - src/titlewidget.cpp
331 fixed: Naming of tracks now Audio 1, Video 1, etc. http://www.kdenlive.org/mantis...
[kdenlive] / src / titlewidget.cpp
index 3c7b15974a5a417310608c546ab8aa6dae3e3ba1..da05ed9f2ec997a2596cbf9ad1b89a087619a16c 100644 (file)
@@ -18,7 +18,7 @@
 #include "titlewidget.h"
 #include "kdenlivesettings.h"
 #include "KoSliderCombo.h"
-
+#include "kthumb.h"
 #include <cmath>
 
 #include <KDebug>
@@ -38,6 +38,8 @@
 #include <QSignalMapper>
 #include <QTextBlockFormat>
 #include <QTextCursor>
+#include <QComboBox>
+#include <QCryptographicHash>
 
 #if QT_VERSION >= 0x040600
 #include <QGraphicsEffect>
@@ -45,6 +47,8 @@
 #include <QGraphicsDropShadowEffect>
 #endif
 
+static QList<TitleTemplate> titletemplates;
+
 int settingUp = false;
 
 const int IMAGEITEM = 7;
@@ -57,15 +61,15 @@ const int SHADOWEFFECT = 2;
 const int TYPEWRITEREFFECT = 3;
 
 TitleWidget::TitleWidget(KUrl url, Timecode tc, QString projectTitlePath, Render *render, QWidget *parent) :
-        QDialog(parent),
-        Ui::TitleWidget_UI(),
-        m_startViewport(NULL),
-        m_endViewport(NULL),
-        m_render(render),
-        m_count(0),
-        m_unicodeDialog(new UnicodeDialog(UnicodeDialog::InputHex)),
-        m_projectTitlePath(projectTitlePath),
-        m_tc(tc)
+    QDialog(parent),
+    Ui::TitleWidget_UI(),
+    m_startViewport(NULL),
+    m_endViewport(NULL),
+    m_render(render),
+    m_count(0),
+    m_unicodeDialog(new UnicodeDialog(UnicodeDialog::InputHex)),
+    m_projectTitlePath(projectTitlePath),
+    m_tc(tc)
 {
     setupUi(this);
     setFont(KGlobalSettings::toolBarFont());
@@ -138,8 +142,10 @@ TitleWidget::TitleWidget(KUrl url, Timecode tc, QString projectTitlePath, Render
     m_frameHeight = render->renderHeight();
     showToolbars(TITLE_SELECT);
 
-    //TODO: get default title duration instead of hardcoded one
-    title_duration->setText(m_tc.getTimecode(GenTime(5000 / 1000.0)));
+    //If project is drop frame, set the input mask as such.
+    title_duration->setInputMask("");
+    title_duration->setValidator(m_tc.validator());
+    title_duration->setText(m_tc.reformatSeparators(KdenliveSettings::title_duration()));
 
     connect(backgroundColor, SIGNAL(clicked()), this, SLOT(slotChangeBackground())) ;
     connect(backgroundAlpha, SIGNAL(valueChanged(qreal, bool)), this, SLOT(slotChangeBackground())) ;
@@ -217,7 +223,6 @@ TitleWidget::TitleWidget(KUrl url, Timecode tc, QString projectTitlePath, Render
     connect(buttonAlignRight, SIGNAL(clicked()), this, SLOT(slotUpdateText()));
     connect(buttonAlignCenter, SIGNAL(clicked()), this, SLOT(slotUpdateText()));
     connect(buttonAlignNone, SIGNAL(clicked()), this, SLOT(slotUpdateText()));
-    //connect(buttonInsertUnicode, SIGNAL(clicked()), this, SLOT(slotInsertUnicode()));
     connect(displayBg, SIGNAL(stateChanged(int)), this, SLOT(displayBackgroundFrame()));
 
     connect(m_unicodeDialog, SIGNAL(charSelected(QString)), this, SLOT(slotInsertUnicodeString(QString)));
@@ -268,13 +273,16 @@ TitleWidget::TitleWidget(KUrl url, Timecode tc, QString projectTitlePath, Render
     zDown->setDefaultAction(m_zDown);
 
     m_zTop = new QAction(KIcon("kdenlive-zindex-top"), QString(), this);
-    m_zTop->setShortcut(Qt::Key_Home);
+    // TODO mbt 1414: Shortcut should change z index only if
+    // cursor is NOT in a text field ...
+    //m_zTop->setShortcut(Qt::Key_Home);
     m_zTop->setToolTip(i18n("Raise object to top"));
     connect(m_zTop, SIGNAL(triggered()), this, SLOT(slotZIndexTop()));
     zTop->setDefaultAction(m_zTop);
 
     m_zBottom = new QAction(KIcon("kdenlive-zindex-bottom"), QString(), this);
-    m_zBottom->setShortcut(Qt::Key_End);
+    // TODO mbt 1414
+    //m_zBottom->setShortcut(Qt::Key_End);
     m_zBottom->setToolTip(i18n("Lower object to bottom"));
     connect(m_zBottom, SIGNAL(triggered()), this, SLOT(slotZIndexBottom()));
     zBottom->setDefaultAction(m_zBottom);
@@ -290,13 +298,13 @@ TitleWidget::TitleWidget(KUrl url, Timecode tc, QString projectTitlePath, Render
     origin_y_top->setToolTip(i18n("Invert y axis and change 0 point"));
     rectBColor->setToolTip(i18n("Select fill color"));
     rectFColor->setToolTip(i18n("Select border color"));
-    rectBAlpha->setToolTip(i18n("Fill transparency"));
-    rectFAlpha->setToolTip(i18n("Border transparency"));
+    rectBAlpha->setToolTip(i18n("Fill opacity"));
+    rectFAlpha->setToolTip(i18n("Border opacity"));
     zoom_slider->setToolTip(i18n("Zoom"));
     buttonRealSize->setToolTip(i18n("Original size (1:1)"));
     buttonFitZoom->setToolTip(i18n("Fit zoom"));
     backgroundColor->setToolTip(i18n("Select background color"));
-    backgroundAlpha->setToolTip(i18n("Background Transparency"));
+    backgroundAlpha->setToolTip(i18n("Background opacity"));
 
     itemhcenter->setIcon(KIcon("kdenlive-align-hor"));
     itemhcenter->setToolTip(i18n("Align item horizontally"));
@@ -414,7 +422,7 @@ TitleWidget::TitleWidget(KUrl url, Timecode tc, QString projectTitlePath, Render
     m_endViewport->setData(0, m_frameWidth);
     m_endViewport->setData(1, m_frameHeight);
 
-    if (!url.isEmpty()) loadTitle(url);
+    if(!url.isEmpty()) loadTitle(url);
     else {
         slotTextTool();
         QTimer::singleShot(200, this, SLOT(slotAdjustZoom()));
@@ -422,8 +430,17 @@ TitleWidget::TitleWidget(KUrl url, Timecode tc, QString projectTitlePath, Render
     initAnimation();
     connect(anim_start, SIGNAL(toggled(bool)), this, SLOT(slotAnimStart(bool)));
     connect(anim_end, SIGNAL(toggled(bool)), this, SLOT(slotAnimEnd(bool)));
+    connect(templateBox, SIGNAL(currentIndexChanged(int)), this, SLOT(templateIndexChanged(int)));
 
     buttonBox->button(QDialogButtonBox::Ok)->setEnabled(KdenliveSettings::hastitleproducer());
+    refreshTitleTemplates();
+    //templateBox->setIconSize(QSize(60,60));
+    templateBox->clear();
+    templateBox->addItem("");
+    foreach(TitleTemplate t, titletemplates) {
+        templateBox->addItem(t.icon, t.name, t.file);
+    }
+    lastDocumentHash = QCryptographicHash::hash(xml().toString().toAscii(), QCryptographicHash::Md5).toHex();
 }
 
 TitleWidget::~TitleWidget()
@@ -458,7 +475,7 @@ QStringList TitleWidget::getFreeTitleInfo(const KUrl &projectUrl, bool isClone)
     titlePath.append((isClone == false) ? "title" : "clone");
     int counter = 0;
     QString path;
-    while (path.isEmpty() || QFile::exists(path)) {
+    while(path.isEmpty() || QFile::exists(path)) {
         counter++;
         path = titlePath + QString::number(counter).rightJustified(3, '0', false) + ".png";
     }
@@ -480,18 +497,65 @@ QString TitleWidget::getTitleResourceFromName(const KUrl &projectUrl, const QStr
 QStringList TitleWidget::extractImageList(QString xml)
 {
     QStringList result;
-    if (xml.isEmpty()) return result;
+    if(xml.isEmpty()) return result;
     QDomDocument doc;
     doc.setContent(xml);
     QDomNodeList images = doc.elementsByTagName("content");
-    for (int i = 0; i < images.count(); i++) {
-        if (images.at(i).toElement().hasAttribute("url"))
+    for(int i = 0; i < images.count(); i++) {
+        if(images.at(i).toElement().hasAttribute("url"))
             result.append(images.at(i).toElement().attribute("url"));
     }
     return result;
 }
 
-
+// static
+QStringList TitleWidget::extractFontList(QString xml)
+{
+    QStringList result;
+    if(xml.isEmpty()) return result;
+    QDomDocument doc;
+    doc.setContent(xml);
+    QDomNodeList images = doc.elementsByTagName("content");
+    for(int i = 0; i < images.count(); i++) {
+        if(images.at(i).toElement().hasAttribute("font"))
+            result.append(images.at(i).toElement().attribute("font"));
+    }
+    return result;
+}
+//static
+void TitleWidget::refreshTitleTemplates()
+{
+    QStringList titlenamelist = QStringList() << i18n("None");
+    QStringList titlefiles = QStringList() << QString();
+    QStringList filters;
+    filters << "*.kdenlivetitle" ;
+    titletemplates.clear();
+    QStringList titleTemplates = KGlobal::dirs()->findDirs("appdata", "titles");
+    foreach(const QString & folder, titleTemplates) {
+        QStringList filesnames = QDir(folder).entryList(filters, QDir::Files);
+        foreach(const QString & fname, filesnames) {
+            //titlenamelist.append(fname);
+            //titlefiles.append(KUrl(folder).path(KUrl::AddTrailingSlash) + fname);
+            TitleTemplate t;
+            t.name = fname;
+            t.file = KUrl(folder).path(KUrl::AddTrailingSlash) + fname;
+            t.icon = QIcon(KThumb::getImage(t.file, 0, 60, 60));
+            titletemplates.append(t);
+        }
+    }
+    kDebug()  << titlenamelist << titlefiles;
+}
+void TitleWidget::templateIndexChanged(int index)
+{
+    QString item = templateBox->itemData(index).toString();
+    if(item != "") {
+        if(lastDocumentHash != QCryptographicHash::hash(xml().toString().toAscii(), QCryptographicHash::Md5).toHex()) {
+            if(KMessageBox::questionYesNo(this, i18n("Do you really want to load a new template? Changes in this title will be lost!")) == KMessageBox::No) return;
+        }
+        loadTitle(item);
+        lastDocumentHash = QCryptographicHash::hash(xml().toString().toAscii(), QCryptographicHash::Md5).toHex();
+    }
+}
 //virtual
 void TitleWidget::resizeEvent(QResizeEvent * /*event*/)
 {
@@ -519,8 +583,8 @@ void TitleWidget::slotSelectTool()
     // Find out which toolbars need to be shown, depending on selected item
     TITLETOOL t = TITLE_SELECT;
     QList<QGraphicsItem *> l = graphicsView->scene()->selectedItems();
-    if (l.size() > 0) {
-        switch (l.at(0)->type()) {
+    if(l.size() > 0) {
+        switch(l.at(0)->type()) {
         case TEXTITEM:
             t = TITLE_TEXT;
             break;
@@ -534,13 +598,13 @@ void TitleWidget::slotSelectTool()
     }
 
     enableToolbars(t);
-    if (t == TITLE_RECTANGLE && (l.at(0) == m_endViewport || l.at(0) == m_startViewport)) {
+    if(t == TITLE_RECTANGLE && (l.at(0) == m_endViewport || l.at(0) == m_startViewport)) {
         //graphicsView->centerOn(l.at(0));
         t = TITLE_SELECT;
     }
     showToolbars(t);
 
-    if (l.size() > 0) {
+    if(l.size() > 0) {
         updateCoordinates(l.at(0));
         updateDimension(l.at(0));
         updateRotZoom(l.at(0));
@@ -554,8 +618,8 @@ void TitleWidget::slotImageTool()
     // TODO: find a way to get a list of all supported image types...
     QString allExtensions = "image/gif image/jpeg image/png image/x-tga image/x-bmp image/svg+xml image/tiff image/x-xcf-gimp image/x-vnd.adobe.photoshop image/x-pcx image/x-exr";
     KUrl url = KFileDialog::getOpenUrl(KUrl(), allExtensions, this, i18n("Load Image")); //"*.svg *.png *.jpg *.jpeg *.gif *.raw"
-    if (!url.isEmpty()) {
-        if (url.path().endsWith(".svg")) {
+    if(!url.isEmpty()) {
+        if(url.path().endsWith(".svg")) {
             QGraphicsSvgItem *svg = new QGraphicsSvgItem(url.toLocalFile());
             svg->setFlags(QGraphicsItem::ItemIsMovable | QGraphicsItem::ItemIsSelectable);
             svg->setZValue(m_count++);
@@ -585,7 +649,7 @@ void TitleWidget::enableToolbars(TITLETOOL toolType)
 {
     // TITLETOOL is defined in graphicsscenerectmove.h
     bool enable = false;
-    if (toolType == TITLE_RECTANGLE || toolType == TITLE_IMAGE) enable = true;
+    if(toolType == TITLE_RECTANGLE || toolType == TITLE_IMAGE) enable = true;
     value_w->setEnabled(enable);
     value_h->setEnabled(enable);
 }
@@ -597,7 +661,7 @@ void TitleWidget::checkButton(TITLETOOL toolType)
     bool bRect = false;
     bool bImage = false;
 
-    switch (toolType) {
+    switch(toolType) {
     case TITLE_SELECT:
         bSelect = true;
         break;
@@ -622,7 +686,7 @@ void TitleWidget::checkButton(TITLETOOL toolType)
 
 void TitleWidget::displayBackgroundFrame()
 {
-    if (!displayBg->isChecked()) {
+    if(!displayBg->isChecked()) {
         QPixmap bg(m_frameWidth / 2, m_frameHeight / 2);
         QPixmap pattern(20, 20);
         pattern.fill();
@@ -678,7 +742,7 @@ void TitleWidget::slotUpdateZoom(int pos)
 void TitleWidget::slotZoom(bool up)
 {
     int pos = zoom_slider->value();
-    if (up) pos++;
+    if(up) pos++;
     else pos--;
     zoom_slider->setValue(pos);
 }
@@ -709,6 +773,7 @@ void TitleWidget::slotNewRect(QGraphicsRectItem * rect)
     f.setAlpha(rectFAlpha->value());
     QPen penf(f);
     penf.setWidth(rectLineWidth->value());
+    penf.setJoinStyle(Qt::RoundJoin);
     rect->setPen(penf);
     QColor b = rectBColor->color();
     b.setAlpha(rectBAlpha->value());
@@ -745,7 +810,7 @@ void TitleWidget::slotNewText(QGraphicsTextItem *tt)
 
     tt->setData(101, outlineWidth);
     tt->setData(102, outlineColor);
-    if (outlineWidth > 0.0) cformat.setTextOutline(QPen(outlineColor, outlineWidth));
+    if(outlineWidth > 0.0) cformat.setTextOutline(QPen(outlineColor, outlineWidth));
 
     cformat.setForeground(QBrush(color));
     cur.setCharFormat(cformat);
@@ -758,7 +823,7 @@ void TitleWidget::slotNewText(QGraphicsTextItem *tt)
 void TitleWidget::setFontBoxWeight(int weight)
 {
     int index = font_weight_box->findData(weight);
-    if (index < 0) {
+    if(index < 0) {
         index = font_weight_box->findData(QFont::Normal);
     }
     font_weight_box->setCurrentIndex(index);
@@ -772,17 +837,18 @@ void TitleWidget::setCurrentItem(QGraphicsItem *item)
 void TitleWidget::zIndexChanged(int v)
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() >= 1) {
+    if(l.size() >= 1) {
         l[0]->setZValue(v);
     }
 }
 
 void TitleWidget::selectionChanged()
 {
-    if (m_scene->tool() != TITLE_SELECT) return;
+    if(m_scene->tool() != TITLE_SELECT) return;
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
     //toolBox->setItemEnabled(2, false);
     //toolBox->setItemEnabled(3, false);
+    effect_list->blockSignals(true);
     value_x->blockSignals(true);
     value_y->blockSignals(true);
     value_w->blockSignals(true);
@@ -791,30 +857,30 @@ void TitleWidget::selectionChanged()
     itemrotatex->blockSignals(true);
     itemrotatey->blockSignals(true);
     itemrotatez->blockSignals(true);
-    if (l.size() == 0) {
+    if(l.size() == 0) {
         effect_stack->setHidden(true);
         effect_frame->setEnabled(false);
         effect_list->setCurrentIndex(0);
         bool blockX = !origin_x_left->signalsBlocked();
         bool blockY = !origin_y_top->signalsBlocked();
-        if (blockX) origin_x_left->blockSignals(true);
-        if (blockY) origin_y_top->blockSignals(true);
+        if(blockX) origin_x_left->blockSignals(true);
+        if(blockY) origin_y_top->blockSignals(true);
         origin_x_left->setChecked(false);
         origin_y_top->setChecked(false);
         updateTextOriginX();
         updateTextOriginY();
         enableToolbars(TITLE_SELECT);
-        if (blockX) origin_x_left->blockSignals(false);
-        if (blockY) origin_y_top->blockSignals(false);
+        if(blockX) origin_x_left->blockSignals(false);
+        if(blockY) origin_y_top->blockSignals(false);
         itemzoom->setEnabled(false);
         itemrotatex->setEnabled(false);
         itemrotatey->setEnabled(false);
         itemrotatez->setEnabled(false);
         frame_properties->setEnabled(false);
-    } else if (l.size() == 1) {
+    } else if(l.size() == 1) {
         effect_frame->setEnabled(true);
         frame_properties->setEnabled(true);
-        if (l.at(0) != m_startViewport && l.at(0) != m_endViewport) {
+        if(l.at(0) != m_startViewport && l.at(0) != m_endViewport) {
             itemzoom->setEnabled(true);
             itemrotatex->setEnabled(true);
             itemrotatey->setEnabled(true);
@@ -826,14 +892,14 @@ void TitleWidget::selectionChanged()
             itemrotatez->setEnabled(false);
             updateInfoText();
         }
-        if (l.at(0)->type() == TEXTITEM) {
+        if(l.at(0)->type() == TEXTITEM) {
             showToolbars(TITLE_TEXT);
             QGraphicsTextItem* i = static_cast <QGraphicsTextItem *>(l.at(0));
-            if (!i->data(100).isNull()) {
+            if(!i->data(100).isNull()) {
                 // Item has an effect
                 QStringList effdata = i->data(100).toStringList();
                 QString effectName = effdata.takeFirst();
-                if (effectName == "typewriter") {
+                if(effectName == "typewriter") {
                     QStringList params = effdata.at(0).split(';');
                     typewriter_delay->setValue(params.at(0).toInt());
                     typewriter_start->setValue(params.at(1).toInt());
@@ -842,16 +908,16 @@ void TitleWidget::selectionChanged()
                 }
             } else {
 #if QT_VERSION >= 0x040600
-                if (i->graphicsEffect()) {
+                if(i->graphicsEffect()) {
                     QGraphicsBlurEffect *blur = static_cast <QGraphicsBlurEffect *>(i->graphicsEffect());
-                    if (blur) {
+                    if(blur) {
                         effect_list->setCurrentIndex(effect_list->findData((int) BLUREFFECT));
                         int rad = (int) blur->blurRadius();
                         blur_radius->setValue(rad);
                         effect_stack->setHidden(false);
                     } else {
                         QGraphicsDropShadowEffect *shad = static_cast <QGraphicsDropShadowEffect *>(i->graphicsEffect());
-                        if (shad) {
+                        if(shad) {
                             effect_list->setCurrentIndex(effect_list->findData((int) SHADOWEFFECT));
                             shadow_radius->setValue(shad->blurRadius());
                             shadow_x->setValue(shad->xOffset());
@@ -860,15 +926,11 @@ void TitleWidget::selectionChanged()
                         }
                     }
                 } else {
-                    effect_list->blockSignals(true);
                     effect_list->setCurrentIndex(effect_list->findData((int) NOEFFECT));
-                    effect_list->blockSignals(false);
                     effect_stack->setHidden(true);
                 }
 #else
-                effect_list->blockSignals(true);
                 effect_list->setCurrentIndex(effect_list->findData((int) NOEFFECT));
-                effect_list->blockSignals(false);
                 effect_stack->setHidden(true);
 #endif
             }
@@ -902,12 +964,12 @@ void TitleWidget::selectionChanged()
             color.setAlpha(255);
             fontColorButton->setColor(color);
 
-            if (!i->data(101).isNull()) {
+            if(!i->data(101).isNull()) {
                 textOutline->blockSignals(true);
-                textOutline->setValue(i->data(101).toDouble()*10);
+                textOutline->setValue(i->data(101).toDouble() * 10);
                 textOutline->blockSignals(false);
             }
-            if (!i->data(102).isNull()) {
+            if(!i->data(102).isNull()) {
                 textOutlineColor->blockSignals(true);
                 textOutlineAlpha->blockSignals(true);
                 color = QColor(i->data(102).toString());
@@ -919,10 +981,10 @@ void TitleWidget::selectionChanged()
             }
             QTextCursor cur = i->textCursor();
             QTextBlockFormat format = cur.blockFormat();
-            if (i->textWidth() == -1) buttonAlignNone->setChecked(true);
-            else if (format.alignment() == Qt::AlignHCenter) buttonAlignCenter->setChecked(true);
-            else if (format.alignment() == Qt::AlignRight) buttonAlignRight->setChecked(true);
-            else if (format.alignment() == Qt::AlignLeft) buttonAlignLeft->setChecked(true);
+            if(i->textWidth() == -1) buttonAlignNone->setChecked(true);
+            else if(format.alignment() == Qt::AlignHCenter) buttonAlignCenter->setChecked(true);
+            else if(format.alignment() == Qt::AlignRight) buttonAlignRight->setChecked(true);
+            else if(format.alignment() == Qt::AlignLeft) buttonAlignLeft->setChecked(true);
 
             font_size->blockSignals(false);
             font_family->blockSignals(false);
@@ -936,16 +998,20 @@ void TitleWidget::selectionChanged()
             buttonAlignNone->blockSignals(false);
             buttonAlignCenter->blockSignals(false);
 
+            // Later
+//      cur.select(QTextCursor::Document);
+//      i->setTextCursor(cur);
+
             updateAxisButtons(i);
             updateCoordinates(i);
             updateDimension(i);
             enableToolbars(TITLE_TEXT);
 
-        } else if ((l.at(0))->type() == RECTITEM) {
+        } else if((l.at(0))->type() == RECTITEM) {
             showToolbars(TITLE_RECTANGLE);
             settingUp = true;
             QGraphicsRectItem *rec = static_cast <QGraphicsRectItem *>(l.at(0));
-            if (rec == m_startViewport || rec == m_endViewport) {
+            if(rec == m_startViewport || rec == m_endViewport) {
                 /*toolBox->setCurrentIndex(3);
                 toolBox->widget(0)->setEnabled(false);
                 toolBox->widget(1)->setEnabled(false);*/
@@ -973,7 +1039,7 @@ void TitleWidget::selectionChanged()
             updateCoordinates(rec);
             updateDimension(rec);
 
-        } else if (l.at(0)->type() == IMAGEITEM) {
+        } else if(l.at(0)->type() == IMAGEITEM) {
             showToolbars(TITLE_IMAGE);
 
             updateCoordinates(l.at(0));
@@ -988,7 +1054,7 @@ void TitleWidget::selectionChanged()
             frame_properties->setEnabled(false);
         }
         zValue->setValue((int)l.at(0)->zValue());
-        if (!l.at(0)->data(ZOOMFACTOR).isNull()) itemzoom->setValue(l.at(0)->data(ZOOMFACTOR).toInt());
+        if(!l.at(0)->data(ZOOMFACTOR).isNull()) itemzoom->setValue(l.at(0)->data(ZOOMFACTOR).toInt());
         else itemzoom->setValue((int)(m_transformations.value(l.at(0)).scalex * 100.0 + 0.5));
         itemrotatex->setValue((int)(m_transformations.value(l.at(0)).rotatex));
         itemrotatey->setValue((int)(m_transformations.value(l.at(0)).rotatey));
@@ -1002,15 +1068,18 @@ void TitleWidget::selectionChanged()
         itemrotatey->blockSignals(false);
         itemrotatez->blockSignals(false);
     }
+    // Tools working on more than one element.
+    if(l.size() > 0)
+        effect_list->blockSignals(false);
 }
 
 void TitleWidget::slotValueChanged(int type)
 {
     QList<QGraphicsItem *> l = graphicsView->scene()->selectedItems();
-    if (l.size() > 0 && l.at(0)->type() == IMAGEITEM) {
+    if(l.size() > 0 && l.at(0)->type() == IMAGEITEM) {
 
         int val = 0;
-        switch (type) {
+        switch(type) {
         case ValueWidth:
             val = value_w->value();
             break;
@@ -1033,7 +1102,7 @@ void TitleWidget::slotValueChanged(int type)
         // Scaling factor
         double scale = 1;
 
-        switch (type) {
+        switch(type) {
         case ValueWidth:
             // Add 0.5 because otherwise incrementing by 1 might have no effect
             length = val / (cos(alpha) + 1 / phi * sin(alpha)) + 0.5;
@@ -1060,28 +1129,25 @@ void TitleWidget::slotValueChanged(int type)
     }
 }
 
-/** \brief Updates position/size of the selected item when a value
- * of an item (coordinates, size) has changed */
 void TitleWidget::slotAdjustSelectedItem()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() >= 1) {
-        if (l.at(0)->type() == RECTITEM) {
+    if(l.size() >= 1) {
+        if(l.at(0)->type() == RECTITEM) {
             //rect item
             QGraphicsRectItem *rec = static_cast <QGraphicsRectItem *>(l.at(0));
             updatePosition(rec);
             rec->setRect(QRect(0, 0, value_w->value(), value_h->value()));
-        } else if (l.at(0)->type() == TEXTITEM) {
+        } else if(l.at(0)->type() == TEXTITEM) {
             //text item
             updatePosition(l.at(0));
-        } else if (l.at(0)->type() == IMAGEITEM) {
+        } else if(l.at(0)->type() == IMAGEITEM) {
             //image item
             updatePosition(l.at(0));
         }
     }
 }
 
-/** \brief Updates width/height int the text fields, regarding transformation matrix */
 void TitleWidget::updateDimension(QGraphicsItem *i)
 {
     value_w->blockSignals(true);
@@ -1089,7 +1155,7 @@ void TitleWidget::updateDimension(QGraphicsItem *i)
     zValue->blockSignals(true);
 
     zValue->setValue((int) i->zValue());
-    if (i->type() == IMAGEITEM) {
+    if(i->type() == IMAGEITEM) {
         // Get multipliers for rotation/scaling
 
         /*Transform t = m_transformations.value(i);
@@ -1101,11 +1167,11 @@ void TitleWidget::updateDimension(QGraphicsItem *i)
 
         value_w->setValue(i->sceneBoundingRect().width());
         value_h->setValue(i->sceneBoundingRect().height());
-    } else if (i->type() == RECTITEM) {
+    } else if(i->type() == RECTITEM) {
         QGraphicsRectItem *r = static_cast <QGraphicsRectItem *>(i);
         value_w->setValue((int) r->rect().width());
         value_h->setValue((int) r->rect().height());
-    } else if (i->type() == TEXTITEM) {
+    } else if(i->type() == TEXTITEM) {
         QGraphicsTextItem *t = static_cast <QGraphicsTextItem *>(i);
         value_w->setValue((int) t->boundingRect().width());
         value_h->setValue((int) t->boundingRect().height());
@@ -1116,19 +1182,18 @@ void TitleWidget::updateDimension(QGraphicsItem *i)
     value_h->blockSignals(false);
 }
 
-/** \brief Updates the coordinates in the text fields from the item */
 void TitleWidget::updateCoordinates(QGraphicsItem *i)
 {
     // Block signals emitted by this method
     value_x->blockSignals(true);
     value_y->blockSignals(true);
 
-    if (i->type() == TEXTITEM) {
+    if(i->type() == TEXTITEM) {
 
         QGraphicsTextItem *rec = static_cast <QGraphicsTextItem *>(i);
 
         // Set the correct x coordinate value
-        if (origin_x_left->isChecked()) {
+        if(origin_x_left->isChecked()) {
             // Origin (0 point) is at m_frameWidth, coordinate axis is inverted
             value_x->setValue((int)(m_frameWidth - rec->pos().x() - rec->boundingRect().width()));
         } else {
@@ -1137,17 +1202,17 @@ void TitleWidget::updateCoordinates(QGraphicsItem *i)
         }
 
         // Same for y
-        if (origin_y_top->isChecked()) {
+        if(origin_y_top->isChecked()) {
             value_y->setValue((int)(m_frameHeight - rec->pos().y() - rec->boundingRect().height()));
         } else {
             value_y->setValue((int) rec->pos().y());
         }
 
-    } else if (i->type() == RECTITEM) {
+    } else if(i->type() == RECTITEM) {
 
         QGraphicsRectItem *rec = static_cast <QGraphicsRectItem *>(i);
 
-        if (origin_x_left->isChecked()) {
+        if(origin_x_left->isChecked()) {
             // Origin (0 point) is at m_frameWidth
             value_x->setValue((int)(m_frameWidth - rec->pos().x() - rec->rect().width()));
         } else {
@@ -1155,21 +1220,21 @@ void TitleWidget::updateCoordinates(QGraphicsItem *i)
             value_x->setValue((int) rec->pos().x());
         }
 
-        if (origin_y_top->isChecked()) {
+        if(origin_y_top->isChecked()) {
             value_y->setValue((int)(m_frameHeight - rec->pos().y() - rec->rect().height()));
         } else {
             value_y->setValue((int) rec->pos().y());
         }
 
-    } else if (i->type() == IMAGEITEM) {
+    } else if(i->type() == IMAGEITEM) {
 
-        if (origin_x_left->isChecked()) {
+        if(origin_x_left->isChecked()) {
             value_x->setValue((int)(m_frameWidth - i->pos().x() - i->sceneBoundingRect().width()));
         } else {
             value_x->setValue((int) i->pos().x());
         }
 
-        if (origin_y_top->isChecked()) {
+        if(origin_y_top->isChecked()) {
             value_y->setValue((int)(m_frameHeight - i->pos().y() - i->sceneBoundingRect().height()));
         } else {
             value_y->setValue((int) i->pos().y());
@@ -1191,7 +1256,7 @@ void TitleWidget::updateRotZoom(QGraphicsItem *i)
 
     Transform t = m_transformations.value(i);
 
-    if (!i->data(ZOOMFACTOR).isNull()) itemzoom->setValue(i->data(ZOOMFACTOR).toInt());
+    if(!i->data(ZOOMFACTOR).isNull()) itemzoom->setValue(i->data(ZOOMFACTOR).toInt());
     else itemzoom->setValue((int)(t.scalex * 100.0 + 0.5));
 
     itemrotatex->setValue((int)(t.rotatex));
@@ -1204,18 +1269,17 @@ void TitleWidget::updateRotZoom(QGraphicsItem *i)
     itemrotatez->blockSignals(false);
 }
 
-/** \brief Updates the position of an item by reading coordinates from the text fields */
 void TitleWidget::updatePosition(QGraphicsItem *i)
 {
-    if (i->type() == TEXTITEM) {
+    if(i->type() == TEXTITEM) {
         QGraphicsTextItem *rec = static_cast <QGraphicsTextItem *>(i);
 
         int posX;
-        if (origin_x_left->isChecked()) {
-            /* Origin of the x axis is at m_frameWidth,
-             * and distance from right border of the item to the right
-             * border of the frame is taken.
-             * See comment to slotOriginXClicked().
+        if(origin_x_left->isChecked()) {
+            /*
+             * Origin of the X axis is at m_frameWidth, and distance from right
+             * border of the item to the right border of the frame is taken. See
+             * comment to slotOriginXClicked().
              */
             posX = m_frameWidth - value_x->value() - rec->boundingRect().width();
         } else {
@@ -1223,7 +1287,7 @@ void TitleWidget::updatePosition(QGraphicsItem *i)
         }
 
         int posY;
-        if (origin_y_top->isChecked()) {
+        if(origin_y_top->isChecked()) {
             /* Same for y axis */
             posY = m_frameHeight - value_y->value() - rec->boundingRect().height();
         } else {
@@ -1232,19 +1296,19 @@ void TitleWidget::updatePosition(QGraphicsItem *i)
 
         rec->setPos(posX, posY);
 
-    } else if (i->type() == RECTITEM) {
+    } else if(i->type() == RECTITEM) {
 
         QGraphicsRectItem *rec = static_cast <QGraphicsRectItem *>(i);
 
         int posX;
-        if (origin_x_left->isChecked()) {
+        if(origin_x_left->isChecked()) {
             posX = m_frameWidth - value_x->value() - rec->rect().width();
         } else {
             posX = value_x->value();
         }
 
         int posY;
-        if (origin_y_top->isChecked()) {
+        if(origin_y_top->isChecked()) {
             posY = m_frameHeight - value_y->value() - rec->rect().height();
         } else {
             posY = value_y->value();
@@ -1252,9 +1316,9 @@ void TitleWidget::updatePosition(QGraphicsItem *i)
 
         rec->setPos(posX, posY);
 
-    } else if (i->type() == IMAGEITEM) {
+    } else if(i->type() == IMAGEITEM) {
         int posX;
-        if (origin_x_left->isChecked()) {
+        if(origin_x_left->isChecked()) {
             // Use the sceneBoundingRect because this also regards transformations like zoom
             posX = m_frameWidth - value_x->value() - i->sceneBoundingRect().width();
         } else {
@@ -1262,7 +1326,7 @@ void TitleWidget::updatePosition(QGraphicsItem *i)
         }
 
         int posY;
-        if (origin_y_top->isChecked()) {
+        if(origin_y_top->isChecked()) {
             posY = m_frameHeight - value_y->value() - i->sceneBoundingRect().height();
         } else {
             posY = value_y->value();
@@ -1276,7 +1340,7 @@ void TitleWidget::updatePosition(QGraphicsItem *i)
 
 void TitleWidget::updateTextOriginX()
 {
-    if (origin_x_left->isChecked()) {
+    if(origin_x_left->isChecked()) {
         origin_x_left->setText(i18n("\u2212X"));
     } else {
         origin_x_left->setText(i18n("+X"));
@@ -1289,7 +1353,7 @@ void TitleWidget::slotOriginXClicked()
     updateTextOriginX();
 
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() >= 1) {
+    if(l.size() >= 1) {
         updateCoordinates(l.at(0));
 
         // Remember x axis setting
@@ -1301,7 +1365,7 @@ void TitleWidget::slotOriginXClicked()
 
 void TitleWidget::updateTextOriginY()
 {
-    if (origin_y_top->isChecked()) {
+    if(origin_y_top->isChecked()) {
         origin_y_top->setText(i18n("\u2212Y"));
     } else {
         origin_y_top->setText(i18n("+Y"));
@@ -1314,7 +1378,7 @@ void TitleWidget::slotOriginYClicked()
     updateTextOriginY();
 
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() >= 1) {
+    if(l.size() >= 1) {
         updateCoordinates(l.at(0));
 
         l.at(0)->setData(TitleDocument::OriginYTop, origin_y_top->isChecked() ?
@@ -1331,14 +1395,14 @@ void TitleWidget::updateAxisButtons(QGraphicsItem *i)
     origin_x_left->blockSignals(true);
     origin_y_top->blockSignals(true);
 
-    if (xAxis == TitleDocument::AxisInverted) {
+    if(xAxis == TitleDocument::AxisInverted) {
         origin_x_left->setChecked(true);
     } else {
         origin_x_left->setChecked(false);
     }
     updateTextOriginX();
 
-    if (yAxis == TitleDocument::AxisInverted) {
+    if(yAxis == TitleDocument::AxisInverted) {
         origin_y_top->setChecked(true);
     } else {
         origin_y_top->setChecked(false);
@@ -1356,41 +1420,36 @@ void TitleWidget::slotChangeBackground()
     m_frameBorder->setBrush(QBrush(color));
 }
 
-/**
- * Something (yeah) has changed in our QGraphicsScene.
- */
 void TitleWidget::slotChanged()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() >= 1 && l.at(0)->type() == TEXTITEM) {
+    if(l.size() >= 1 && l.at(0)->type() == TEXTITEM) {
         textChanged(static_cast <QGraphicsTextItem *>(l.at(0)));
     }
 }
 
-/**
- * If the user has set origin_x_left (everything also for y),
- * we need to look whether a text element has been selected. If yes,
- * we need to ensure that the right border of the text field
- * remains fixed also when some text has been entered.
- *
- * This is also known as right-justified, with the difference that
- * it is not valid for text but for its boundingRect. Text may still
- * be left-justified.
- */
 void TitleWidget::textChanged(QGraphicsTextItem *i)
 {
-
+    /*
+     * If the user has set origin_x_left (the same for y), we need to look
+     * whether a text element has been selected. If yes, we need to ensure that
+     * the right border of the text field remains fixed also when some text has
+     * been entered.
+     *
+     * This is also known as right-justified, with the difference that it is not
+     * valid for text but for its boundingRect. Text may still be
+     * left-justified.
+     */
     updateDimension(i);
 
-    if (origin_x_left->isChecked() || origin_y_top->isChecked()) {
-
-        if (!i->toPlainText().isEmpty()) {
+    if(origin_x_left->isChecked() || origin_y_top->isChecked()) {
+        if(!i->toPlainText().isEmpty())
             updatePosition(i);
-        else {
+        else {
             /*
-             * Don't do anything if the string is empty. If the position
-             * would be updated here, a newly created text field would
-             * be set to the position of the last selected text field.
+             * Don't do anything if the string is empty. If the position were
+             * updated here, a newly created text field would be set to the
+             * position of the last selected text field.
              */
         }
     }
@@ -1404,8 +1463,8 @@ void TitleWidget::slotInsertUnicode()
 void TitleWidget::slotInsertUnicodeString(QString text)
 {
     QList<QGraphicsItem *> l = graphicsView->scene()->selectedItems();
-    if (l.size() > 0) {
-        if (l.at(0)->type() == TEXTITEM) {
+    if(l.size() > 0) {
+        if(l.at(0)->type() == TEXTITEM) {
             QGraphicsTextItem *t = static_cast <QGraphicsTextItem *>(l.at(0));
             t->textCursor().insertText(text);
         }
@@ -1427,19 +1486,19 @@ void TitleWidget::slotUpdateText()
     double outlineWidth = textOutline->value() / 10.0;
     QGraphicsTextItem* item = NULL;
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1 && l.at(0)->type() == TEXTITEM) {
+    if(l.size() == 1 && l.at(0)->type() == TEXTITEM) {
         item = static_cast <QGraphicsTextItem *>(l.at(0));
     }
-    if (!item) return;
+    if(!item) return;
     //if (item->textCursor().selection ().isEmpty())
     QTextCursor cur(item->document());
     cur.select(QTextCursor::Document);
     QTextBlockFormat format = cur.blockFormat();
-    if (buttonAlignLeft->isChecked() || buttonAlignCenter->isChecked() || buttonAlignRight->isChecked()) {
+    if(buttonAlignLeft->isChecked() || buttonAlignCenter->isChecked() || buttonAlignRight->isChecked()) {
         item->setTextWidth(item->boundingRect().width());
-        if (buttonAlignCenter->isChecked()) format.setAlignment(Qt::AlignHCenter);
-        else if (buttonAlignRight->isChecked()) format.setAlignment(Qt::AlignRight);
-        else if (buttonAlignLeft->isChecked()) format.setAlignment(Qt::AlignLeft);
+        if(buttonAlignCenter->isChecked()) format.setAlignment(Qt::AlignHCenter);
+        else if(buttonAlignRight->isChecked()) format.setAlignment(Qt::AlignRight);
+        else if(buttonAlignLeft->isChecked()) format.setAlignment(Qt::AlignLeft);
     } else {
         format.setAlignment(Qt::AlignLeft);
         item->setTextWidth(-1);
@@ -1450,7 +1509,7 @@ void TitleWidget::slotUpdateText()
 
     item->setData(101, outlineWidth);
     item->setData(102, outlineColor);
-    if (outlineWidth > 0.0) cformat.setTextOutline(QPen(outlineColor, outlineWidth, Qt::SolidLine, Qt::RoundCap, Qt::RoundJoin));
+    if(outlineWidth > 0.0) cformat.setTextOutline(QPen(outlineColor, outlineWidth, Qt::SolidLine, Qt::RoundCap, Qt::RoundJoin));
 
     cformat.setForeground(QBrush(color));
     cur.setCharFormat(cformat);
@@ -1463,12 +1522,13 @@ void TitleWidget::slotUpdateText()
 void TitleWidget::rectChanged()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1 && l.at(0)->type() == RECTITEM && !settingUp) {
+    if(l.size() == 1 && l.at(0)->type() == RECTITEM && !settingUp) {
         QGraphicsRectItem *rec = static_cast<QGraphicsRectItem *>(l.at(0));
         QColor f = rectFColor->color();
         f.setAlpha(rectFAlpha->value());
         QPen penf(f);
         penf.setWidth(rectLineWidth->value());
+        penf.setJoinStyle(Qt::RoundJoin);
         rec->setPen(penf);
         QColor b = rectBColor->color();
         b.setAlpha(rectBAlpha->value());
@@ -1479,7 +1539,7 @@ void TitleWidget::rectChanged()
 void TitleWidget::itemScaled(int val)
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1) {
+    if(l.size() == 1) {
         Transform x = m_transformations.value(l.at(0));
         x.scalex = (double)val / 100.0;
         x.scaley = (double)val / 100.0;
@@ -1513,9 +1573,9 @@ void TitleWidget::itemRotateZ(qreal val)
 void TitleWidget::itemRotate(qreal val, int axis)
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1) {
+    if(l.size() == 1) {
         Transform x = m_transformations[l.at(0)];
-        switch (axis) {
+        switch(axis) {
         case 0:
             x.rotatex = val;
             break;
@@ -1536,7 +1596,7 @@ void TitleWidget::itemRotate(qreal val, int axis)
         qtrans.rotate(x.rotatez, Qt::ZAxis);
         l[0]->setTransform(qtrans);
         m_transformations[l.at(0)] = x;
-        if (l[0]->data(ZOOMFACTOR).isNull()) l[0]->setData(ZOOMFACTOR, 100);
+        if(l[0]->data(ZOOMFACTOR).isNull()) l[0]->setData(ZOOMFACTOR, 100);
         updateDimension(l.at(0));
     }
 }
@@ -1544,7 +1604,7 @@ void TitleWidget::itemRotate(qreal val, int axis)
 void TitleWidget::itemHCenter()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1) {
+    if(l.size() == 1) {
         QGraphicsItem *item = l.at(0);
         QRectF br = item->sceneBoundingRect();
         int width = (int)br.width();
@@ -1558,7 +1618,7 @@ void TitleWidget::itemHCenter()
 void TitleWidget::itemVCenter()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1) {
+    if(l.size() == 1) {
         QGraphicsItem *item = l.at(0);
         QRectF br = item->sceneBoundingRect();
         int height = (int)br.height();
@@ -1572,11 +1632,11 @@ void TitleWidget::itemVCenter()
 void TitleWidget::itemTop()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1) {
+    if(l.size() == 1) {
         QGraphicsItem *item = l.at(0);
         QRectF br = item->sceneBoundingRect();
         double diff;
-        if (br.top() > 0) diff = -br.top();
+        if(br.top() > 0) diff = -br.top();
         else diff = -br.bottom();
         item->moveBy(0, diff);
         updateCoordinates(item);
@@ -1586,11 +1646,11 @@ void TitleWidget::itemTop()
 void TitleWidget::itemBottom()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1) {
+    if(l.size() == 1) {
         QGraphicsItem *item = l.at(0);
         QRectF br = item->sceneBoundingRect();
         double diff;
-        if (br.bottom() > m_frameHeight) diff = m_frameHeight - br.top();
+        if(br.bottom() > m_frameHeight) diff = m_frameHeight - br.top();
         else diff = m_frameHeight - br.bottom();
         item->moveBy(0, diff);
         updateCoordinates(item);
@@ -1600,11 +1660,11 @@ void TitleWidget::itemBottom()
 void TitleWidget::itemLeft()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1) {
+    if(l.size() == 1) {
         QGraphicsItem *item = l.at(0);
         QRectF br = item->sceneBoundingRect();
         double diff;
-        if (br.left() > 0) diff = -br.left();
+        if(br.left() > 0) diff = -br.left();
         else diff = -br.right();
         item->moveBy(diff, 0);
         updateCoordinates(item);
@@ -1614,11 +1674,11 @@ void TitleWidget::itemLeft()
 void TitleWidget::itemRight()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1) {
+    if(l.size() == 1) {
         QGraphicsItem *item = l.at(0);
         QRectF br = item->sceneBoundingRect();
         double diff;
-        if (br.right() < m_frameWidth) diff = m_frameWidth - br.right();
+        if(br.right() < m_frameWidth) diff = m_frameWidth - br.right();
         else diff = m_frameWidth - br.left();
         item->moveBy(diff, 0);
         updateCoordinates(item);
@@ -1651,19 +1711,19 @@ void TitleWidget::setupViewports()
 
 void TitleWidget::loadTitle(KUrl url)
 {
-    if (url.isEmpty()) url = KFileDialog::getOpenUrl(KUrl(m_projectTitlePath), "application/x-kdenlivetitle", this, i18n("Load Title"));
-    if (!url.isEmpty()) {
+    if(url.isEmpty()) url = KFileDialog::getOpenUrl(KUrl(m_projectTitlePath), "application/x-kdenlivetitle", this, i18n("Load Title"));
+    if(!url.isEmpty()) {
         QList<QGraphicsItem *> items = m_scene->items();
-        for (int i = 0; i < items.size(); i++) {
-            if (items.at(i)->zValue() > -1000) delete items.at(i);
+        for(int i = 0; i < items.size(); i++) {
+            if(items.at(i)->zValue() > -1000) delete items.at(i);
         }
         m_scene->clearTextSelection();
         QDomDocument doc;
         QString tmpfile;
 
-        if (KIO::NetAccess::download(url, tmpfile, 0)) {
+        if(KIO::NetAccess::download(url, tmpfile, 0)) {
             QFile file(tmpfile);
-            if (file.open(QIODevice::ReadOnly)) {
+            if(file.open(QIODevice::ReadOnly)) {
                 doc.setContent(&file, false);
                 file.close();
             } else return;
@@ -1691,9 +1751,9 @@ void TitleWidget::loadTitle(KUrl url)
 
 void TitleWidget::saveTitle(KUrl url)
 {
-    if (anim_start->isChecked()) slotAnimStart(false);
-    if (anim_end->isChecked()) slotAnimEnd(false);
-    if (url.isEmpty()) {
+    if(anim_start->isChecked()) slotAnimStart(false);
+    if(anim_end->isChecked()) slotAnimEnd(false);
+    if(url.isEmpty()) {
         KFileDialog *fs = new KFileDialog(KUrl(m_projectTitlePath), "application/x-kdenlivetitle", this);
         fs->setOperationMode(KFileDialog::Saving);
         fs->setMode(KFile::File);
@@ -1705,8 +1765,8 @@ void TitleWidget::saveTitle(KUrl url)
         url = fs->selectedUrl();
         delete fs;
     }
-    if (!url.isEmpty()) {
-        if (m_titledocument.saveDocument(url, m_startViewport, m_endViewport, m_tc.getFrameCount(title_duration->text())) == false)
+    if(!url.isEmpty()) {
+        if(m_titledocument.saveDocument(url, m_startViewport, m_endViewport, m_tc.getFrameCount(title_duration->text()) - 1) == false)
             KMessageBox::error(this, i18n("Cannot write to file %1", url.path()));
     }
 }
@@ -1714,13 +1774,13 @@ void TitleWidget::saveTitle(KUrl url)
 QDomDocument TitleWidget::xml()
 {
     QDomDocument doc = m_titledocument.xml(m_startViewport, m_endViewport);
-    doc.documentElement().setAttribute("out", m_tc.getFrameCount(title_duration->text()));
+    doc.documentElement().setAttribute("out", m_tc.getFrameCount(title_duration->text()) - 1);
     return doc;
 }
 
-int TitleWidget::duration() const
+int TitleWidget::outPoint() const
 {
-    return m_tc.getFrameCount(title_duration->text());
+    return m_tc.getFrameCount(title_duration->text()) - 1;
 }
 
 void TitleWidget::setXml(QDomDocument doc)
@@ -1728,7 +1788,7 @@ void TitleWidget::setXml(QDomDocument doc)
     int out;
     m_count = m_titledocument.loadFromXml(doc, m_startViewport, m_endViewport, &out);
     adjustFrameSize();
-    title_duration->setText(m_tc.getTimecode(GenTime(out, m_render->fps())));
+    title_duration->setText(m_tc.getTimecode(GenTime(out + 1, m_render->fps())));
     /*if (doc.documentElement().hasAttribute("out")) {
     GenTime duration = GenTime(doc.documentElement().attribute("out").toDouble() / 1000.0);
     title_duration->setText(m_tc.getTimecode(duration));
@@ -1739,22 +1799,22 @@ void TitleWidget::setXml(QDomDocument doc)
     m_transformations.clear();
     QList <QGraphicsItem *> items = graphicsView->scene()->items();
     const double PI = 4.0 * atan(1.0);
-    for (int i = 0; i < items.count(); i++) {
+    for(int i = 0; i < items.count(); i++) {
         QTransform t = items.at(i)->transform();
         Transform x;
         x.scalex = t.m11();
         x.scaley = t.m22();
-        if (!items.at(i)->data(ROTATEFACTOR).isNull()) {
+        if(!items.at(i)->data(ROTATEFACTOR).isNull()) {
             QList<QVariant> rotlist = items.at(i)->data(ROTATEFACTOR).toList();
-            if (rotlist.count() >= 3) {
+            if(rotlist.count() >= 3) {
                 x.rotatex = rotlist[0].toDouble();
                 x.rotatey = rotlist[1].toDouble();
                 x.rotatez = rotlist[2].toDouble();
 
                 // Try to adjust zoom
-                t.rotate(x.rotatex * (-1), Qt::XAxis);
-                t.rotate(x.rotatey * (-1), Qt::YAxis);
-                t.rotate(x.rotatez * (-1), Qt::ZAxis);
+                t.rotate(x.rotatex *(-1), Qt::XAxis);
+                t.rotate(x.rotatey *(-1), Qt::YAxis);
+                t.rotate(x.rotatez *(-1), Qt::ZAxis);
                 x.scalex = t.m11();
                 x.scaley = t.m22();
             } else {
@@ -1791,15 +1851,13 @@ void TitleWidget::setXml(QDomDocument doc)
     selectionChanged();
 }
 
-/** \brief Connected to the accepted signal - calls writeChoices */
 void TitleWidget::slotAccepted()
 {
-    if (anim_start->isChecked()) slotAnimStart(false);
-    if (anim_end->isChecked()) slotAnimEnd(false);
+    if(anim_start->isChecked()) slotAnimStart(false);
+    if(anim_end->isChecked()) slotAnimEnd(false);
     writeChoices();
 }
 
-/** \brief Store the current choices of font, background and rect values */
 void TitleWidget::writeChoices()
 {
     // Get a pointer to a shared configuration instance, then get the TitleWidget group.
@@ -1832,7 +1890,6 @@ void TitleWidget::writeChoices()
 
 }
 
-/** \brief Read the last stored choices into the dialog */
 void TitleWidget::readChoices()
 {
     // Get a pointer to a shared configuration instance, then get the TitleWidget group.
@@ -1850,7 +1907,7 @@ void TitleWidget::readChoices()
     textOutline->setValue(titleConfig.readEntry("font_outline", textOutline->value()));
 
     int weight;
-    if (titleConfig.readEntry("font_bold", false)) weight = QFont::Bold;
+    if(titleConfig.readEntry("font_bold", false)) weight = QFont::Bold;
     else weight = titleConfig.readEntry("font_weight", font_weight_box->itemData(font_weight_box->currentIndex()).toInt());
     setFontBoxWeight(weight);
     buttonItalic->setChecked(titleConfig.readEntry("font_italic", buttonItalic->isChecked()));
@@ -1876,15 +1933,15 @@ void TitleWidget::adjustFrameSize()
 
 void TitleWidget::slotAnimStart(bool anim)
 {
-    if (anim && anim_end->isChecked()) {
+    if(anim && anim_end->isChecked()) {
         anim_end->setChecked(false);
         m_endViewport->setZValue(-1000);
         m_endViewport->setBrush(QBrush());
     }
     slotSelectTool();
     QList<QGraphicsItem *> list = m_scene->items();
-    for (int i = 0; i < list.count(); i++) {
-        if (list.at(i)->zValue() > -1000) {
+    for(int i = 0; i < list.count(); i++) {
+        if(list.at(i)->zValue() > -1000) {
             list.at(i)->setFlag(QGraphicsItem::ItemIsMovable, !anim);
             list.at(i)->setFlag(QGraphicsItem::ItemIsSelectable, !anim);
         }
@@ -1896,7 +1953,7 @@ void TitleWidget::slotAnimStart(bool anim)
     itemrotatez->setEnabled(!anim);
     frame_toolbar->setEnabled(!anim);
     toolbar_stack->setEnabled(!anim);
-    if (anim) {
+    if(anim) {
         keep_aspect->setChecked(!m_startViewport->data(0).isNull());
         m_startViewport->setZValue(1100);
         QColor col = m_startViewport->pen().color();
@@ -1906,27 +1963,27 @@ void TitleWidget::slotAnimStart(bool anim)
         m_startViewport->setSelected(true);
         selectionChanged();
         slotSelectTool();
-        if (m_startViewport->childItems().isEmpty()) addAnimInfoText();
+        if(m_startViewport->childItems().isEmpty()) addAnimInfoText();
     } else {
         m_startViewport->setZValue(-1000);
         m_startViewport->setBrush(QBrush());
         m_startViewport->setFlags(0);
-        if (!anim_end->isChecked()) deleteAnimInfoText();
+        if(!anim_end->isChecked()) deleteAnimInfoText();
     }
 
 }
 
 void TitleWidget::slotAnimEnd(bool anim)
 {
-    if (anim && anim_start->isChecked()) {
+    if(anim && anim_start->isChecked()) {
         anim_start->setChecked(false);
         m_startViewport->setZValue(-1000);
         m_startViewport->setBrush(QBrush());
     }
     slotSelectTool();
     QList<QGraphicsItem *> list = m_scene->items();
-    for (int i = 0; i < list.count(); i++) {
-        if (list.at(i)->zValue() > -1000) {
+    for(int i = 0; i < list.count(); i++) {
+        if(list.at(i)->zValue() > -1000) {
             list.at(i)->setFlag(QGraphicsItem::ItemIsMovable, !anim);
             list.at(i)->setFlag(QGraphicsItem::ItemIsSelectable, !anim);
         }
@@ -1938,7 +1995,7 @@ void TitleWidget::slotAnimEnd(bool anim)
     itemrotatez->setEnabled(!anim);
     frame_toolbar->setEnabled(!anim);
     toolbar_stack->setEnabled(!anim);
-    if (anim) {
+    if(anim) {
         keep_aspect->setChecked(!m_endViewport->data(0).isNull());
         m_endViewport->setZValue(1100);
         QColor col = m_endViewport->pen().color();
@@ -1948,12 +2005,12 @@ void TitleWidget::slotAnimEnd(bool anim)
         m_endViewport->setSelected(true);
         selectionChanged();
         slotSelectTool();
-        if (m_endViewport->childItems().isEmpty()) addAnimInfoText();
+        if(m_endViewport->childItems().isEmpty()) addAnimInfoText();
     } else {
         m_endViewport->setZValue(-1000);
         m_endViewport->setBrush(QBrush());
         m_endViewport->setFlags(0);
-        if (!anim_start->isChecked()) deleteAnimInfoText();
+        if(!anim_start->isChecked()) deleteAnimInfoText();
     }
 }
 
@@ -1978,17 +2035,17 @@ void TitleWidget::addAnimInfoText()
 void TitleWidget::updateInfoText()
 {
     // update info text font
-    if (!m_startViewport->childItems().isEmpty()) {
+    if(!m_startViewport->childItems().isEmpty()) {
         QGraphicsTextItem *item = static_cast <QGraphicsTextItem *>(m_startViewport->childItems().at(0));
-        if (item) {
+        if(item) {
             QFont font = item->font();
             font.setPixelSize(m_startViewport->rect().width() / 10);
             item->setFont(font);
         }
     }
-    if (!m_endViewport->childItems().isEmpty()) {
+    if(!m_endViewport->childItems().isEmpty()) {
         QGraphicsTextItem *item = static_cast <QGraphicsTextItem *>(m_endViewport->childItems().at(0));
-        if (item) {
+        if(item) {
             QFont font = item->font();
             font.setPixelSize(m_endViewport->rect().width() / 10);
             item->setFont(font);
@@ -1999,22 +2056,21 @@ void TitleWidget::updateInfoText()
 void TitleWidget::deleteAnimInfoText()
 {
     // end animation editing, remove info text
-    while (!m_startViewport->childItems().isEmpty()) {
+    while(!m_startViewport->childItems().isEmpty()) {
         QGraphicsItem *item = m_startViewport->childItems().at(0);
         m_scene->removeItem(item);
         delete item;
     }
-    while (!m_endViewport->childItems().isEmpty()) {
+    while(!m_endViewport->childItems().isEmpty()) {
         QGraphicsItem *item = m_endViewport->childItems().at(0);
         m_scene->removeItem(item);
         delete item;
     }
 }
 
-
 void TitleWidget::slotKeepAspect(bool keep)
 {
-    if (m_endViewport->zValue() == 1100) {
+    if(m_endViewport->zValue() == 1100) {
         m_endViewport->setData(0, keep == true ? m_frameWidth : QVariant());
         m_endViewport->setData(1, keep == true ? m_frameHeight : QVariant());
     } else {
@@ -2025,78 +2081,87 @@ void TitleWidget::slotKeepAspect(bool keep)
 
 void TitleWidget::slotResize50()
 {
-    if (m_endViewport->zValue() == 1100) {
+    if(m_endViewport->zValue() == 1100) {
         m_endViewport->setRect(0, 0, m_frameWidth / 2, m_frameHeight / 2);
     } else m_startViewport->setRect(0, 0, m_frameWidth / 2, m_frameHeight / 2);
 }
 
 void TitleWidget::slotResize100()
 {
-    if (m_endViewport->zValue() == 1100) {
+    if(m_endViewport->zValue() == 1100) {
         m_endViewport->setRect(0, 0, m_frameWidth, m_frameHeight);
     } else m_startViewport->setRect(0, 0, m_frameWidth, m_frameHeight);
 }
 
 void TitleWidget::slotResize200()
 {
-    if (m_endViewport->zValue() == 1100) {
+    if(m_endViewport->zValue() == 1100) {
         m_endViewport->setRect(0, 0, m_frameWidth * 2, m_frameHeight * 2);
     } else m_startViewport->setRect(0, 0, m_frameWidth * 2, m_frameHeight * 2);
 }
 
 void TitleWidget::slotAddEffect(int ix)
 {
-    QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
+    QList<QGraphicsItem *> list = graphicsView->scene()->selectedItems();
     int effect = effect_list->itemData(ix).toInt();
-    if (effect == NOEFFECT) {
-        if (l.size() == 1) l[0]->setData(100, QVariant());
-        effect_stack->setHidden(true);
-        return;
-    }
-    effect_stack->setCurrentIndex(effect - 1);
-    effect_stack->setHidden(false);
-    if (effect == TYPEWRITEREFFECT) {
-        if (l.size() == 1 && l.at(0)->type() == TEXTITEM) {
-            QStringList effdata = QStringList() << "typewriter" << QString::number(typewriter_delay->value()) + ";" + QString::number(typewriter_start->value());
-            l[0]->setData(100, effdata);
-        }
-    }
-#if QT_VERSION < 0x040600
-    return;
-#else
-    if (effect == BLUREFFECT) {
-        // Blur effect
-        if (l.size() == 1) {
-            QGraphicsEffect *eff = new QGraphicsBlurEffect();
-            l[0]->setGraphicsEffect(eff);
-        }
-    } else if (effect == SHADOWEFFECT) {
-        if (l.size() == 1) {
-            QGraphicsEffect *eff = new QGraphicsDropShadowEffect();
-            l[0]->setGraphicsEffect(eff);
+
+    if(list.size() == 1) {
+        if(effect == NOEFFECT)
+            effect_stack->setHidden(true);
+        else {
+            effect_stack->setCurrentIndex(effect - 1);
+            effect_stack->setHidden(false);
         }
-    }
+    } else // Hide the effects stack when more than one element is selected.
+        effect_stack->setHidden(true);
 
+    foreach(QGraphicsItem * item, list) {
+        switch(effect) {
+        case NOEFFECT:
+            item->setData(100, QVariant());
+#if QT_VERSION >= 0x040600
+            item->setGraphicsEffect(0);
 #endif
+            break;
+        case TYPEWRITEREFFECT:
+            /*
+             * Allow the user to set the typewriter effect to more than one
+             * element, but do not add it to non-text elements.
+             */
+            if(item->type() == TEXTITEM) {
+                QStringList effdata = QStringList() << "typewriter" << QString::number(typewriter_delay->value()) + ";" + QString::number(typewriter_start->value());
+                item->setData(100, effdata);
+            }
+            break;
+#if QT_VERSION >= 0x040600
+            // Do not remove the non-QGraphicsEffects.
+        case BLUREFFECT:
+            item->setGraphicsEffect(new QGraphicsBlurEffect());
+            break;
+        case SHADOWEFFECT:
+            item->setGraphicsEffect(new QGraphicsDropShadowEffect());
+            break;
+#endif
+        }
+    }
 }
 
-
 void TitleWidget::slotFontText(const QString& s)
 {
     const QFont f(s);
-    if (f.exactMatch()) {
+    if(f.exactMatch()) {
         // Font really exists (could also just be a «d» if the user
         // starts typing «dejavu» for example).
         font_family->setCurrentFont(f);
     }
     // Note: Typing dejavu serif does not recognize the font (takes sans)
-    // in older Qt versions.
+    // in older Qt versions. Case must match there (except for first letter)
 }
 
-void TitleWidget::slotEditTypewriter(int ix)
+void TitleWidget::slotEditTypewriter(int /*ix*/)
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1) {
+    if(l.size() == 1) {
         QStringList effdata = QStringList() << "typewriter" << QString::number(typewriter_delay->value()) + ";" + QString::number(typewriter_start->value());
         l[0]->setData(100, effdata);
     }
@@ -2108,10 +2173,10 @@ void TitleWidget::slotEditBlur(int ix)
     return;
 #else
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1) {
+    if(l.size() == 1) {
         QGraphicsEffect *eff = l[0]->graphicsEffect();
         QGraphicsBlurEffect *blur = static_cast <QGraphicsBlurEffect *>(eff);
-        if (blur) blur->setBlurRadius(ix);
+        if(blur) blur->setBlurRadius(ix);
     }
 #endif
 }
@@ -2122,10 +2187,10 @@ void TitleWidget::slotEditShadow()
     return;
 #else
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() == 1) {
+    if(l.size() == 1) {
         QGraphicsEffect *eff = l[0]->graphicsEffect();
         QGraphicsDropShadowEffect *shadow = static_cast <QGraphicsDropShadowEffect *>(eff);
-        if (shadow) {
+        if(shadow) {
             shadow->setBlurRadius(shadow_radius->value());
             shadow->setOffset(shadow_x->value(), shadow_y->value());
         }
@@ -2133,27 +2198,33 @@ void TitleWidget::slotEditShadow()
 #endif
 }
 
-qreal TitleWidget::zIndexBounds(bool maxBound)
+qreal TitleWidget::zIndexBounds(bool maxBound, bool intersectingOnly)
 {
     qreal bound = maxBound ? -99 : 99;
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() > 0) {
-        QList<QGraphicsItem*> lItems = graphicsView->scene()->items(l[0]->sceneBoundingRect(), Qt::IntersectsItemShape);
-        if (lItems.size() > 0) {
+    if(l.size() > 0) {
+        QList<QGraphicsItem*> lItems;
+        // Get items (all or intersecting only)
+        if(intersectingOnly) {
+            lItems = graphicsView->scene()->items(l[0]->sceneBoundingRect(), Qt::IntersectsItemShape);
+        } else {
+            lItems = graphicsView->scene()->items();
+        }
+        if(lItems.size() > 0) {
             int n = lItems.size();
             qreal z;
-            if (maxBound) {
-                for (int i = 0; i < n; i++) {
+            if(maxBound) {
+                for(int i = 0; i < n; i++) {
                     z = lItems[i]->zValue();
-                    if (z > bound && !lItems[i]->isSelected()) {
+                    if(z > bound && !lItems[i]->isSelected()) {
                         bound = z;
                     }
                 }
             } else {
                 // Get minimum z index.
-                for (int i = 0; i < n; i++) {
+                for(int i = 0; i < n; i++) {
                     z = lItems[i]->zValue();
-                    if (z < bound && !lItems[i]->isSelected() && z > -999) {
+                    if(z < bound && !lItems[i]->isSelected() && z > -999) {
                         // There are items at the very bottom (background e.g.) with z-index < -1000.
                         bound = z;
                     }
@@ -2167,10 +2238,10 @@ qreal TitleWidget::zIndexBounds(bool maxBound)
 void TitleWidget::slotZIndexUp()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() >= 1) {
+    if(l.size() >= 1) {
         qreal currentZ = l[0]->zValue();
-        qreal max = zIndexBounds(true);
-        if (currentZ <= max) {
+        qreal max = zIndexBounds(true, true);
+        if(currentZ <= max) {
             l[0]->setZValue(currentZ + 1);
             updateDimension(l[0]);
         }
@@ -2180,10 +2251,10 @@ void TitleWidget::slotZIndexUp()
 void TitleWidget::slotZIndexTop()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() >= 1) {
+    if(l.size() >= 1) {
         qreal currentZ = l[0]->zValue();
-        qreal max = zIndexBounds(true);
-        if (currentZ <= max) {
+        qreal max = zIndexBounds(true, false);
+        if(currentZ <= max) {
             l[0]->setZValue(max + 1);
             updateDimension(l[0]);
         }
@@ -2193,10 +2264,10 @@ void TitleWidget::slotZIndexTop()
 void TitleWidget::slotZIndexDown()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() >= 1) {
+    if(l.size() >= 1) {
         qreal currentZ = l[0]->zValue();
-        qreal min = zIndexBounds(false);
-        if (currentZ >= min) {
+        qreal min = zIndexBounds(false, true);
+        if(currentZ >= min) {
             l[0]->setZValue(currentZ - 1);
             updateDimension(l[0]);
         }
@@ -2206,10 +2277,10 @@ void TitleWidget::slotZIndexDown()
 void TitleWidget::slotZIndexBottom()
 {
     QList<QGraphicsItem*> l = graphicsView->scene()->selectedItems();
-    if (l.size() >= 1) {
+    if(l.size() >= 1) {
         qreal currentZ = l[0]->zValue();
-        qreal min = zIndexBounds(false);
-        if (currentZ >= min) {
+        qreal min = zIndexBounds(false, false);
+        if(currentZ >= min) {
             l[0]->setZValue(min - 1);
             updateDimension(l[0]);
         }