[Kdenlive-devel] Please restore pre-0.9.10 fade on AV tracks

Karl Kiniger karl.kiniger at med.ge.com
Fri Oct 17 14:46:39 UTC 2014


Please.

Reason: in my workflow with AV tracks audio fades I need fade from/to black very 
        rarely. However on many simple edit tasks which dont justify splitting A/V
	into V and A  audio fades are often necessary to avoid noticeable glitches.

Thank you very much writing/improving/maintaining kdenlive,

Karl 

diff --git a/src/timeline/customtrackview.cpp b/src/timeline/customtrackview.cpp
index 58ae93a..6356e0e 100644
--- a/src/timeline/customtrackview.cpp
+++ b/src/timeline/customtrackview.cpp
@@ -4100,7 +4100,7 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
         // new fade in
         } else if (item->fadeIn() != 0) {
             QDomElement effect;
-            if (item->isVideoOnly() || (item->clipType() != Audio && !item->isAudioOnly() && item->clipType() != Playlist)) {
+            if (item->isVideoOnly() || (((item->clipType() != Audio) && (item->clipType()) != AV && (item->clipType() != Playlist)))) {
                 effect = MainWindow::videoEffects.getEffectByTag("", "fade_from_black").cloneNode().toElement();
             } else {
                 effect = MainWindow::audioEffects.getEffectByTag("volume", "fadein").cloneNode().toElement();
@@ -4133,7 +4133,7 @@ void CustomTrackView::mouseReleaseEvent(QMouseEvent * event)
         // new fade out
         } else if (item->fadeOut() != 0) {
             QDomElement effect;
-            if (item->isVideoOnly() || (item->clipType() != Audio && !item->isAudioOnly() && item->clipType() != Playlist)) {
+            if (item->isVideoOnly() || (((item->clipType() != Audio) && (item->clipType()) != AV && (item->clipType() != Playlist)))) {
                 effect = MainWindow::videoEffects.getEffectByTag("", "fade_to_black").cloneNode().toElement();
             } else {
                 effect = MainWindow::audioEffects.getEffectByTag("volume", "fadeout").cloneNode().toElement();







More information about the Kdenlive mailing list