I’m generally ok with this (squash the waveform and draw text above it), the only objection I see is that you lose precious vertical space to show a label that may be unhelpful (people often just have the audio clip that corresponds to the video clip, with very little ambiguity)
Unless you are doing extremely basic video editing, you are always going to have various additional audio tracks for background music and sound effects that all run parallel to your video tracks.
Personally, I do a lot of lipsync work for games, and I find myself juggling with tons of short voice snippets in the sequencer. I constantly need to know the file names and exact amplitudes of all of them.
Most other NLEs aloocate name space above the Thumbnail/waveform.
Thumbs/waves can be toggled shut/collapsed to preserve vertical space. But the name is always apparent, although I cannot think of any apps that bother showing the path as well as the asset name?
Yeah, showing the entire asset path on every strip is definitely a bit much. It’s especially pointless when the path is way longer than the strip itself which is often the case.
If its so important to a user perhaps the path could appear as a tooltip? As it stands you cannot copy the path from timeline anyway so its only for reference.
Unfortunately I haven’t had time to improve this patch.
Some level of refactoring is in progress in the VSE code, maybe I’ll have another go when the code is more stable (for instance after D3934 is merged).
Does this include any of the strip preview code? More specifically, the waveform preview plus audio filename?
Someone needs to tell me. I am unable to read any of this code. (Let alone write code.)
@Gabs this is only the cache code, @mathers mentioned that they would hopefully get to work on the strip preview code again once this code was committed.
Awesome !
This is great news. Unfortunately I won’t have time to do another round before a few weeks, but I’ll definitely give it another go.
Another great thing that was a pleasant surprise to me : the scale of the audio waveform in the sequencer is now linked to the strip’s audio volume !