X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;ds=sidebyside;f=src%2Fclipproperties.cpp;h=82c536f15508a9ba3300b7f0c747af5ee9fe739c;hb=25c924be85d1b49c78d71977d29b2a4383904d9e;hp=7981f9537cb666953525e5fcc801e6b24da59875;hpb=4191df91ef00b8e1ea8f5cf627da46eb53246302;p=kdenlive diff --git a/src/clipproperties.cpp b/src/clipproperties.cpp index 7981f953..82c536f1 100644 --- a/src/clipproperties.cpp +++ b/src/clipproperties.cpp @@ -27,6 +27,7 @@ #include #include #include +#include #include #include #include @@ -55,14 +56,15 @@ static const int METATAB = 6; static const int ADVANCEDTAB = 7; -ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidget * parent) : - QDialog(parent), - m_clip(clip), - m_tc(tc), - m_fps(fps), - m_count(0), - m_clipNeedsRefresh(false), - m_clipNeedsReLoad(false) +ClipProperties::ClipProperties(DocClipBase *clip, const Timecode &tc, double fps, QWidget * parent) : + QDialog(parent) + , m_clip(clip) + , m_tc(tc) + , m_fps(fps) + , m_count(0) + , m_clipNeedsRefresh(false) + , m_clipNeedsReLoad(false) + , m_proxyContainer(NULL) { setAttribute(Qt::WA_DeleteOnClose, true); setFont(KGlobalSettings::toolBarFont()); @@ -83,7 +85,7 @@ ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidg // New display aspect ratio support if (props.contains("force_aspect_num") && props.value("force_aspect_num").toInt() > 0 && - props.contains("force_aspect_den") && props.value("force_aspect_den").toInt() > 0) { + props.contains("force_aspect_den") && props.value("force_aspect_den").toInt() > 0) { m_view.clip_force_ar->setChecked(true); m_view.clip_ar_num->setEnabled(true); m_view.clip_ar_den->setEnabled(true); @@ -123,14 +125,16 @@ ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidg } connect(m_view.clip_force_framerate, SIGNAL(toggled(bool)), this, SLOT(slotModified())); connect(m_view.clip_framerate, SIGNAL(valueChanged(double)), this, SLOT(slotModified())); + m_view.clip_progressive->addItem(i18n("Interlaced"), 0); + m_view.clip_progressive->addItem(i18n("Progressive"), 1); if (props.contains("force_progressive")) { m_view.clip_force_progressive->setChecked(true); m_view.clip_progressive->setEnabled(true); - m_view.clip_progressive->setValue(props.value("force_progressive").toInt()); + m_view.clip_progressive->setCurrentIndex(props.value("force_progressive").toInt()); } connect(m_view.clip_force_progressive, SIGNAL(toggled(bool)), this, SLOT(slotModified())); - connect(m_view.clip_progressive, SIGNAL(valueChanged(int)), this, SLOT(slotModified())); + connect(m_view.clip_progressive, SIGNAL(currentIndexChanged(int)), this, SLOT(slotModified())); m_view.clip_fieldorder->addItem(i18n("Bottom first"), 0); m_view.clip_fieldorder->addItem(i18n("Top first"), 1); @@ -193,12 +197,42 @@ ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidg connect(m_view.clip_full_luma, SIGNAL(toggled(bool)), this, SLOT(slotModified())); // Check for Metadata - QMap meta = m_clip->metadata(); - QMap::const_iterator i = meta.constBegin(); + QMap meta = m_clip->metadata(); + QMap::const_iterator i = meta.constBegin(); while (i != meta.constEnd()) { - QTreeWidgetItem *metaitem = new QTreeWidgetItem(m_view.metadata_list); + QStringList values = i.value(); + QString parentName; + QString iconName; + if (values.count() > 1 && !values.at(1).isEmpty()) parentName = values.at(1); + else { + if (KdenliveSettings::ffmpegpath().endsWith("avconv")) { + parentName = i18n("Libav"); + iconName = "meta_libav.png"; + } + else { + parentName = i18n("FFmpeg"); + iconName = "meta_ffmpeg.png"; + } + } + QTreeWidgetItem *parent = NULL; + QList matches = m_view.metadata_list->findItems(parentName, Qt::MatchExactly); + if (!matches.isEmpty()) parent = matches.at(0); + else { + if (parentName == "Magic Lantern") iconName = "meta_magiclantern.png"; + parent = new QTreeWidgetItem(m_view.metadata_list, QStringList() << parentName); + if (!iconName.isEmpty()) { + KIcon icon(KStandardDirs::locate("appdata", iconName)); + parent->setIcon(0, icon); + } + } + QTreeWidgetItem *metaitem = NULL; + if (parent) { + metaitem = new QTreeWidgetItem(parent); + parent->setExpanded(true); + } + else metaitem = new QTreeWidgetItem(m_view.metadata_list); metaitem->setText(0, i.key()); //i18n(i.key().section('.', 2, 3).toUtf8().data())); - metaitem->setText(1, i.value()); + metaitem->setText(1, values.at(0)); ++i; } @@ -261,7 +295,7 @@ ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidg } if (t == PLAYLIST) - m_view.tabWidget->setTabText(VIDEOTAB, i18n("Playlist")); + m_view.tabWidget->setTabText(VIDEOTAB, i18n("Playlist")); if (t == IMAGE) { m_view.tabWidget->removeTab(SLIDETAB); @@ -331,7 +365,7 @@ ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidg m_view.animation->setCurrentIndex(0); QString path = props.value("resource"); QString ext = path.section('.', -1); - for (int i = 0; i < m_view.image_type->count(); i++) { + for (int i = 0; i < m_view.image_type->count(); ++i) { if (m_view.image_type->itemData(i).toString() == ext) { m_view.image_type->setCurrentIndex(i); break; @@ -345,7 +379,7 @@ ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidg m_view.slide_duration_frames->setHidden(true); m_view.luma_duration_frames->setHidden(true); - parseFolder(); + parseFolder(false); m_view.luma_duration->setText(tc.getTimecodeFromFrames(props.value("luma_duration").toInt())); QString lumaFile = props.value("luma_file"); @@ -408,48 +442,10 @@ ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidg m_view.clip_aproperties->setItemDelegate(del2); m_view.clip_aproperties->setStyleSheet(QString("QTreeWidget { background-color: transparent;}")); m_view.clip_vproperties->setStyleSheet(QString("QTreeWidget { background-color: transparent;}")); - - if (props.contains("videocodec")) - new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Video codec") << props.value("videocodec")); - - if (props.contains("frame_size")) - new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Frame size") << props.value("frame_size")); - - if (props.contains("fps")) { - new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Frame rate") << props.value("fps")); - if (!m_view.clip_framerate->isEnabled()) m_view.clip_framerate->setValue(props.value("fps").toDouble()); - } - - if (props.contains("progressive")) { - int scanning = props.value("progressive").toInt(); - QString txt = scanning == 1 ? i18n("Progressive") : i18n("Interlaced"); - new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Scanning") << txt); - } + loadVideoProperties(props); - if (props.contains("aspect_ratio")) - new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Pixel aspect ratio") << props.value("aspect_ratio")); - - if (props.contains("pix_fmt")) - new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Pixel format") << props.value("pix_fmt")); - - if (props.contains("colorspace")) - new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Colorspace") << ProfilesDialog::getColorspaceDescription(props.value("colorspace").toInt())); + m_view.clip_thumb->setMinimumSize(180 * KdenliveSettings::project_display_ratio(), 180); - - int width = 180.0 * KdenliveSettings::project_display_ratio(); - if (width % 2 == 1) width++; - QPixmap pix = m_clip->thumbProducer()->getImage(url, m_clip->getClipThumbFrame(), width, 180); - QPixmap framedPix(pix.width(), pix.height()); - framedPix.fill(Qt::transparent); - QPainter p(&framedPix); - p.setRenderHint(QPainter::Antialiasing, true); - QPainterPath path; - path.addRoundedRect(0.5, 0.5, framedPix.width() - 1, framedPix.height() - 1, 4, 4); - p.setClipPath(path); - p.drawPixmap(0, 0, pix); - p.end(); - - m_view.clip_thumb->setPixmap(framedPix); if (t == IMAGE || t == VIDEO || t == PLAYLIST) m_view.tabWidget->removeTab(AUDIOTAB); } else { m_view.tabWidget->removeTab(IMAGETAB); @@ -492,7 +488,7 @@ ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidg m_view.analysis_save->setIcon(KIcon("document-save-as")); m_view.analysis_save->setToolTip(i18n("Save analysis data")); - // Check for Nepomuk metadata + // Check for Nepomuk metadata #ifdef USE_NEPOMUK #if KDE_IS_VERSION(4,6,0) if (!url.isEmpty()) { @@ -504,7 +500,7 @@ ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidg m_view.clip_license->setText(i18n("License: %1", res.property(Nepomuk::Vocabulary::NIE::license()).toString())); if (ltype.startsWith("http")) { m_view.clip_license->setUrl(ltype); - connect(m_view.clip_license, SIGNAL(leftClickedUrl(const QString &)), this, SLOT(slotOpenUrl(const QString &))); + connect(m_view.clip_license, SIGNAL(leftClickedUrl(QString)), this, SLOT(slotOpenUrl(QString))); } } else m_view.clip_license->setHidden(true); @@ -525,7 +521,7 @@ ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidg connect(m_view.marker_delete, SIGNAL(clicked()), this, SLOT(slotDeleteMarker())); connect(m_view.marker_save, SIGNAL(clicked()), this, SLOT(slotSaveMarkers())); connect(m_view.marker_load, SIGNAL(clicked()), this, SLOT(slotLoadMarkers())); - connect(m_view.markers_list, SIGNAL(doubleClicked(const QModelIndex &)), this, SLOT(slotEditMarker())); + connect(m_view.markers_list, SIGNAL(doubleClicked(QModelIndex)), this, SLOT(slotEditMarker())); connect(m_view.analysis_delete, SIGNAL(clicked()), this, SLOT(slotDeleteAnalysis())); connect(m_view.analysis_save, SIGNAL(clicked()), this, SLOT(slotSaveAnalysis())); @@ -534,12 +530,16 @@ ClipProperties::ClipProperties(DocClipBase *clip, Timecode tc, double fps, QWidg connect(this, SIGNAL(accepted()), this, SLOT(slotApplyProperties())); connect(m_view.buttonBox->button(QDialogButtonBox::Apply), SIGNAL(clicked()), this, SLOT(slotApplyProperties())); m_view.buttonBox->button(QDialogButtonBox::Apply)->setEnabled(false); + + m_view.metadata_list->resizeColumnToContents(0); + m_view.clip_vproperties->resizeColumnToContents(0); + m_view.clip_aproperties->resizeColumnToContents(0); adjustSize(); } // Used for multiple clips editing -ClipProperties::ClipProperties(QList cliplist, Timecode tc, QMap commonproperties, QWidget * parent) : +ClipProperties::ClipProperties(const QList &cliplist, const Timecode &tc, const QMap &commonproperties, QWidget * parent) : QDialog(parent), m_clip(NULL), m_tc(tc), @@ -573,7 +573,7 @@ ClipProperties::ClipProperties(QList cliplist, Timecode tc, QMap if (commonproperties.contains("force_progressive") && !commonproperties.value("force_progressive").isEmpty()) { m_view.clip_force_progressive->setChecked(true); m_view.clip_progressive->setEnabled(true); - m_view.clip_progressive->setValue(commonproperties.value("force_progressive").toInt()); + m_view.clip_progressive->setCurrentIndex(commonproperties.value("force_progressive").toInt()); } if (commonproperties.contains("force_tff") && !commonproperties.value("force_tff").isEmpty()) { @@ -683,9 +683,57 @@ ClipProperties::ClipProperties(QList cliplist, Timecode tc, QMap ClipProperties::~ClipProperties() { QAbstractItemDelegate *del1 = m_view.clip_vproperties->itemDelegate(); - if (del1) delete del1; + delete del1; QAbstractItemDelegate *del2 = m_view.clip_aproperties->itemDelegate(); - if (del2) delete del2; + delete del2; +} + + +void ClipProperties::loadVideoProperties(const QMap &props) +{ + m_view.clip_vproperties->clear(); + if (props.contains("videocodec")) + new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Video codec") << props.value("videocodec")); + else if (props.contains("videocodecid")) + new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Video codec") << props.value("videocodecid")); + + if (props.contains("frame_size")) + new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Frame size") << props.value("frame_size")); + + if (props.contains("fps")) { + new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Frame rate") << props.value("fps")); + if (!m_view.clip_framerate->isEnabled()) m_view.clip_framerate->setValue(props.value("fps").toDouble()); + } + + if (props.contains("progressive")) { + int scanning = props.value("progressive").toInt(); + QString txt = scanning == 1 ? i18n("Progressive") : i18n("Interlaced"); + new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Scanning") << txt); + } + + if (props.contains("aspect_ratio")) + new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Pixel aspect ratio") << props.value("aspect_ratio")); + + if (props.contains("pix_fmt")) + new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Pixel format") << props.value("pix_fmt")); + + if (props.contains("colorspace")) + new QTreeWidgetItem(m_view.clip_vproperties, QStringList() << i18n("Colorspace") << ProfilesDialog::getColorspaceDescription(props.value("colorspace").toInt())); +} + +void ClipProperties::slotGotThumbnail(const QString &id, const QImage &img) +{ + if (id != m_clip->getId()) return; + QPixmap framedPix(img.width(), img.height()); + framedPix.fill(Qt::transparent); + QPainter p(&framedPix); + p.setRenderHint(QPainter::Antialiasing, true); + QPainterPath path; + path.addRoundedRect(0.5, 0.5, framedPix.width() - 1, framedPix.height() - 1, 4, 4); + p.setClipPath(path); + p.drawImage(0, 0, img); + p.end(); + m_view.clip_thumb->setPixmap(framedPix); } void ClipProperties::slotApplyProperties() @@ -693,10 +741,25 @@ void ClipProperties::slotApplyProperties() if (m_clip != NULL) { QMap props = properties(); emit applyNewClipProperties(m_clip->getId(), m_clip->currentProperties(props), props, needsTimelineRefresh(), needsTimelineReload()); + QTimer::singleShot(1000, this, SLOT(slotReloadVideoProperties())); + if (props.contains("force_aspect_num")) + QTimer::singleShot(1000, this, SLOT(slotReloadVideoThumb())); } m_view.buttonBox->button(QDialogButtonBox::Apply)->setEnabled(false); } +void ClipProperties::slotReloadVideoProperties() +{ + if (m_clip == NULL) return; + loadVideoProperties(m_clip->properties()); +} + +void ClipProperties::slotReloadVideoThumb() +{ + if (m_clip == NULL) return; + emit requestThumb(QString('?' + m_clip->getId()), QList() << m_clip->getClipThumbFrame()); +} + void ClipProperties::disableClipId(const QString &id) { if (m_clip && m_view.buttonBox->button(QDialogButtonBox::Ok)->isEnabled()) { @@ -744,10 +807,10 @@ void ClipProperties::slotUpdateAnalysisData(DocClipBase *clip) m_view.analysis_box->setHidden(analysis.isEmpty()); QMap::const_iterator i = analysis.constBegin(); while (i != analysis.constEnd()) { - QStringList itemtext; - itemtext << i.key() << i.value(); - (void) new QTreeWidgetItem(m_view.analysis_list, itemtext); - ++i; + QStringList itemtext; + itemtext << i.key() << i.value(); + (void) new QTreeWidgetItem(m_view.analysis_list, itemtext); + ++i; } } @@ -761,7 +824,7 @@ void ClipProperties::slotFillMarkersList(DocClipBase *clip) QStringList itemtext; itemtext << time << marks.at(count).comment(); QTreeWidgetItem *item = new QTreeWidgetItem(m_view.markers_list, itemtext); - item->setData(0, Qt::DecorationRole, CommentedTime::markerColor(marks.at(count).markerType())); + item->setData(0, Qt::DecorationRole, CommentedTime::markerColor(marks.at(count).markerType())); } } @@ -769,9 +832,11 @@ void ClipProperties::slotAddMarker() { CommentedTime marker(GenTime(), i18n("Marker")); QPointer d = new MarkerDialog(m_clip, marker, - m_tc, i18n("Add Marker"), this); + m_tc, i18n("Add Marker"), this); if (d->exec() == QDialog::Accepted) { - emit addMarker(m_clip->getId(), d->newMarker()); + QList markers; + markers << d->newMarker(); + emit addMarkers(m_clip->getId(), markers); } delete d; } @@ -793,7 +858,9 @@ void ClipProperties::slotEditMarker() if (pos < 0 || pos > marks.count() - 1) return; MarkerDialog d(m_clip, marks.at(pos), m_tc, i18n("Edit Marker"), this); if (d.exec() == QDialog::Accepted) { - emit addMarker(m_clip->getId(), d.newMarker()); + QList markers; + markers << d.newMarker(); + emit addMarkers(m_clip->getId(), markers); } } @@ -801,21 +868,45 @@ void ClipProperties::slotDeleteMarker() { QList < CommentedTime > marks = m_clip->commentedSnapMarkers(); QList < CommentedTime > toDelete; - for (int i = 0; i < marks.count(); i++) { - if (m_view.markers_list->topLevelItem(i)->isSelected()) { - CommentedTime marker = marks.at(i); - marker.setMarkerType(-1); - toDelete << marker; - } + for (int i = 0; i < marks.count(); ++i) { + if (m_view.markers_list->topLevelItem(i)->isSelected()) { + CommentedTime marker = marks.at(i); + marker.setMarkerType(-1); + toDelete << marker; + } } - for (int i = 0; i < toDelete.count(); i++) - emit addMarker(m_clip->getId(), toDelete.at(i)); + emit addMarkers(m_clip->getId(), toDelete); } void ClipProperties::slotDeleteAnalysis() { QTreeWidgetItem *current = m_view.analysis_list->currentItem(); - if (current) emit deleteAnalysis(m_clip->getId(), current->text(0)); + if (current) emit editAnalysis(m_clip->getId(), current->text(0), QString()); +} + +void ClipProperties::slotSaveAnalysis() +{ + QString url = KFileDialog::getSaveFileName(KUrl("kfiledialog:///projectfolder"), "text/plain", this, i18n("Save Analysis Data")); + if (url.isEmpty()) return; + KSharedConfigPtr config = KSharedConfig::openConfig(url, KConfig::SimpleConfig); + KConfigGroup analysisConfig(config, "Analysis"); + QTreeWidgetItem *current = m_view.analysis_list->currentItem(); + analysisConfig.writeEntry(current->text(0), current->text(1)); +} + +void ClipProperties::slotLoadAnalysis() +{ + QString url = KFileDialog::getOpenFileName(KUrl("kfiledialog:///projectfolder"), "text/plain", this, i18n("Open Analysis Data")); + if (url.isEmpty()) return; + KSharedConfigPtr config = KSharedConfig::openConfig(url, KConfig::SimpleConfig); + KConfigGroup transConfig(config, "Analysis"); + // read the entries + QMap< QString, QString > profiles = transConfig.entryMap(); + QMapIterator i(profiles); + while (i.hasNext()) { + i.next(); + emit editAnalysis(m_clip->getId(), i.key(), i.value()); + } } const QString &ClipProperties::clipId() const @@ -823,7 +914,6 @@ const QString &ClipProperties::clipId() const return m_clip->getId(); } - QMap ClipProperties::properties() { QMap props; @@ -838,7 +928,7 @@ QMap ClipProperties::properties() int aspectDenominator = m_view.clip_ar_den->value(); if (m_view.clip_force_ar->isChecked()) { if (aspectNumerator != m_old_props.value("force_aspect_num").toInt() || - aspectDenominator != m_old_props.value("force_aspect_den").toInt()) { + aspectDenominator != m_old_props.value("force_aspect_den").toInt()) { props["force_aspect_num"] = QString::number(aspectNumerator); props["force_aspect_den"] = QString::number(aspectDenominator); props["force_aspect_ratio"].clear(); @@ -866,9 +956,9 @@ QMap ClipProperties::properties() m_clipNeedsRefresh = true; } - int progressive = m_view.clip_progressive->value(); + int progressive = m_view.clip_progressive->currentIndex(); if (m_view.clip_force_progressive->isChecked()) { - if (progressive != m_old_props.value("force_progressive").toInt()) { + if (!m_old_props.contains("force_progressive") || progressive != m_old_props.value("force_progressive").toInt()) { props["force_progressive"] = QString::number(progressive); } } else if (m_old_props.contains("force_progressive") && !m_old_props.value("force_progressive").isEmpty()) { @@ -877,7 +967,7 @@ QMap ClipProperties::properties() int fieldOrder = m_view.clip_fieldorder->currentIndex(); if (m_view.clip_force_fieldorder->isChecked()) { - if (fieldOrder != m_old_props.value("force_tff").toInt()) { + if (!m_old_props.contains("force_tff") || fieldOrder != m_old_props.value("force_tff").toInt()) { props["force_tff"] = QString::number(fieldOrder); } } else if (m_old_props.contains("force_tff") && !m_old_props.value("force_tff").isEmpty()) { @@ -1050,7 +1140,7 @@ bool ClipProperties::needsTimelineReload() const } -void ClipProperties::parseFolder() +void ClipProperties::parseFolder(bool reloadThumb) { QString path = m_view.clip_path->text(); bool isMime = !(path.contains('%')); @@ -1070,15 +1160,29 @@ void ClipProperties::parseFolder() QStringList result = dir.entryList(QDir::Files); if (!isMime) { - // find pattern - QString filter = KUrl(m_view.clip_path->text()).fileName(); + int offset = 0; + QString path = m_view.clip_path->text(); + if (path.contains('?')) { + // New MLT syntax + offset = m_view.clip_path->text().section(':', -1).toInt(); + path = path.section('?', 0, 0); + } + QString filter = KUrl(path).fileName(); QString ext = filter.section('.', -1); filter = filter.section('%', 0, -2); QString regexp = '^' + filter + "\\d+\\." + ext + '$'; QRegExp rx(regexp); QStringList entries; + int ix; foreach(const QString & path, result) { - if (rx.exactMatch(path)) entries << path; + if (rx.exactMatch(path)) { + if (offset > 0) { + // make sure our image is in the range we want (> begin) + ix = path.section(filter, 1).section('.', 0, 0).toInt(); + if (ix < offset) continue; + } + entries << path; + } } result = entries; } @@ -1093,16 +1197,16 @@ void ClipProperties::parseFolder() } m_view.buttonBox->button(QDialogButtonBox::Ok)->setEnabled(true); m_view.slide_info->setText(i18np("1 image found", "%1 images found", m_count)); - QDomElement xml = m_clip->toXML(); - xml.setAttribute("resource", m_view.clip_path->text() + extension); - int width = 180.0 * KdenliveSettings::project_display_ratio(); - if (width % 2 == 1) width++; - QString filePath = m_view.clip_path->text(); - if (isMime) filePath.append(extension); - QPixmap pix = m_clip->thumbProducer()->getImage(KUrl(filePath), 1, width, 180); QMap props = m_clip->properties(); m_view.clip_duration->setText(m_tc.getTimecodeFromFrames(props.value("ttl").toInt() * m_count)); - m_view.clip_thumb->setPixmap(pix); + if (reloadThumb) { + int width = 180.0 * KdenliveSettings::project_display_ratio(); + if (width % 2 == 1) width++; + QString filePath = m_view.clip_path->text(); + if (isMime) filePath.append(extension); + QPixmap pix = m_clip->thumbProducer()->getImage(KUrl(filePath), 1, width, 180); + m_view.clip_thumb->setPixmap(pix); + } } void ClipProperties::slotCheckMaxLength() @@ -1138,10 +1242,10 @@ void ClipProperties::slotUpdateDurationFormat(int ix) void ClipProperties::slotDeleteProxy() { - QString proxy = m_clip->getProperty("proxy"); - if (proxy.isEmpty()) return; - emit deleteProxy(proxy); - if (m_proxyContainer) delete m_proxyContainer; + QString proxy = m_clip->getProperty("proxy"); + if (proxy.isEmpty()) return; + emit deleteProxy(proxy); + if (m_proxyContainer) delete m_proxyContainer; } void ClipProperties::slotOpenUrl(const QString &url)