From: <tt...@us...> - 2010-10-24 11:23:51
|
Revision: 5041 http://kdenlive.svn.sourceforge.net/kdenlive/?rev=5041&view=rev Author: ttill Date: 2010-10-24 11:23:44 +0000 (Sun, 24 Oct 2010) Log Message: ----------- Fix values for multiple parameters are all the same when adding a keyframe after the last one in the timeline Modified Paths: -------------- trunk/kdenlive/src/clipitem.cpp Modified: trunk/kdenlive/src/clipitem.cpp =================================================================== --- trunk/kdenlive/src/clipitem.cpp 2010-10-24 11:01:23 UTC (rev 5040) +++ trunk/kdenlive/src/clipitem.cpp 2010-10-24 11:23:44 UTC (rev 5041) @@ -1699,7 +1699,12 @@ added = true; } else newkfr.append(str); } - if (!added) newkfr.append(QString::number(pos) + ":" + QString::number(val)); + if (!added) { + if (i == 0) + newkfr.append(QString::number(pos) + ":" + QString::number(val)); + else + newkfr.append(QString::number(pos) + ":" + e.attribute("default")); + } e.setAttribute("keyframes", newkfr.join(";")); } } This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <tt...@us...> - 2011-04-20 10:58:25
|
Revision: 5542 http://kdenlive.svn.sourceforge.net/kdenlive/?rev=5542&view=rev Author: ttill Date: 2011-04-20 10:58:18 +0000 (Wed, 20 Apr 2011) Log Message: ----------- Automatically crop relative to the project resolution on clips using proxies: http://www.kdenlive.org/mantis/view.php?id=2056 Modified Paths: -------------- trunk/kdenlive/src/clipitem.cpp Modified: trunk/kdenlive/src/clipitem.cpp =================================================================== --- trunk/kdenlive/src/clipitem.cpp 2011-04-20 09:48:20 UTC (rev 5541) +++ trunk/kdenlive/src/clipitem.cpp 2011-04-20 10:58:18 UTC (rev 5542) @@ -248,6 +248,12 @@ } } + if (effect.attribute("id") == "crop") { + // default use_profile to 1 for clips with proxies to avoid problems when rendering + if (e.attribute("name") == "use_profile" && !(m_clip->getProperty("proxy").isEmpty() || m_clip->getProperty("proxy") == "-")) + e.setAttribute("value", "1"); + } + if ((e.attribute("type") == "keyframe" || e.attribute("type") == "simplekeyframe") && e.attribute("keyframes").isEmpty()) { // Effect has a keyframe type parameter, we need to set the values e.setAttribute("keyframes", QString::number(cropStart().frames(m_fps)) + ':' + e.attribute("default")); This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <tt...@us...> - 2010-05-06 07:24:36
|
Revision: 4412 http://kdenlive.svn.sourceforge.net/kdenlive/?rev=4412&view=rev Author: ttill Date: 2010-05-06 07:24:29 +0000 (Thu, 06 May 2010) Log Message: ----------- Allow resizing images in timeline from start: http://kdenlive.org/mantis/view.php?id=1575 Modified Paths: -------------- trunk/kdenlive/src/clipitem.cpp Modified: trunk/kdenlive/src/clipitem.cpp =================================================================== --- trunk/kdenlive/src/clipitem.cpp 2010-05-05 00:06:00 UTC (rev 4411) +++ trunk/kdenlive/src/clipitem.cpp 2010-05-06 07:24:29 UTC (rev 4412) @@ -1143,7 +1143,17 @@ void ClipItem::resizeStart(int posx) { const int min = (startPos() - cropStart()).frames(m_fps); - if (posx < min) posx = min; + if (posx < min) { + if (clipType() == IMAGE || clipType() == COLOR || clipType() == TEXT) { + GenTime oldPos = startPos(); + moveBy(-1 * (oldPos - GenTime(posx, m_fps)).frames(m_fps), 0); + const int newPos = (endPos() + (oldPos - startPos())).frames(m_fps); + AbstractClipItem::resizeEnd(newPos); + } + posx = min; + } + + if (posx == startPos().frames(m_fps)) return; const int previous = cropStart().frames(m_fps); AbstractClipItem::resizeStart(posx); This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: Alberto V. <av...@fr...> - 2010-05-06 08:29:10
Attachments:
signature.asc
|
On Thursday 06 May 2010 09:24:29 tt...@us... wrote: > - if (posx < min) posx = min; > + if (posx < min) { > + if (clipType() == IMAGE || clipType() == COLOR || clipType() == > TEXT) { + GenTime oldPos = startPos(); > + moveBy(-1 * (oldPos - GenTime(posx, m_fps)).frames(m_fps), 0); > + const int newPos = (endPos() + (oldPos - > startPos())).frames(m_fps); + > AbstractClipItem::resizeEnd(newPos); > + } > + posx = min; is this correct? shouldn't posx be positioned to the new leftmost position? -- Alberto Villa, FreeBSD committer <avilla@FreeBSD.org> http://people.FreeBSD.org/~avilla Too clever is dumb. -- Ogden Nash |
From: j-b-m <j-...@us...> - 2010-05-06 10:05:12
|
On Thursday 06 May 2010 10:28:52 Alberto Villa wrote: > On Thursday 06 May 2010 09:24:29 tt...@us... wrote: > > - if (posx < min) posx = min; > > + if (posx < min) { > > + if (clipType() == IMAGE || clipType() == COLOR || clipType() == > > TEXT) { + GenTime oldPos = startPos(); > > + moveBy(-1 * (oldPos - GenTime(posx, m_fps)).frames(m_fps), > > 0); + const int newPos = (endPos() + (oldPos - > > startPos())).frames(m_fps); + > > AbstractClipItem::resizeEnd(newPos); > > + } > > + posx = min; > > is this correct? shouldn't posx be positioned to the new leftmost position? I just committed some changes that will hopefully correctly fix the original issue: http://kdenlive.org/mantis/view.php?id=1575 regards jb |