This patch adds the ability to delete the Proxy of the active sequencer strip.
This action can not be undone, so confirmation dialog must be used.
"All proxy files for this strip will be permanently removed. Do you want to continue?"
Or something in that note.
It doesn't seem, that this operator requires invoke method, it could be removed.
Please use a bit more descriptive names
also descriptive names
I mean it may be correct naming scheme, but it doesn't look right in context of this file. Other editors have this quite mixed, so I don't know. Don't want to nitpick too much, but on the other hand I want to point to potential issues...
Maybe it is time to rename these operators to be in consistency with 2.8 class naming?
SequencerCrossfadeSounds, SequencerCutMulticam, SequencerDeinterlaceSelectedMovies, SequencerFadesClear, SequencerFadesAdd,
Digging through WM API, I realized, that the invoke method, was actually used as a confirmation.
It was quite poor in execution, so it would be better if we could explain to user, what is going to happen. Not quite sure if that is possible
This doesn't seem necessary or consistent with other operators.
Nearly any operator that uses the selection could also use the active-only, however we don't provide this option.
Poll function should not loop over selection, it's likely to backfire when drawing the interface and slow down interaction.
Also, was this tested? it's accessing self which isn't defined in classmethods.
This copy-pastes code from clip.py, did you look into sharing logic between these two functions? It seems like this is a case it could be done.
The built-in proxy function allows for both building of the active strip and building of proxies for selected strips. Currently there is a Build Proxies function in the menu - so there should also be a delete proxies function. In operators executed from the VSE Menus pretty much all operate on selection, whereas operators executed from the Strip tab in the sidebar are typically only affecting the active strip unless +alt is pressed. If the following reworking of the current proxy panel, which separates the proxy functions into Active in the Strip panel and Selection in the Proxies & Cache tab, should be accepted, then separated delete proxy/proxies will also be necessary: https://developer.blender.org/D6077
So for those reasons, I suggested that Eitan added options for both active and selected strips.
For the moment you could make a staticmethod on the class CLIP_OT_delete_proxy in clip.py, import and call that.
Later we could move to ./release/scripts/modules/bpy_extras/video_utils.py.