This comes handy!

Question is if it wouldn't be better to do timing conversion in do_versions and not add "Correct Timing" toggle to UI.
Would make UI and code a bit cleaner.

Is there any sensible reason to have such toggle?

In D4072#92489, @Richard Antalik (ISS) wrote:

Question is if it wouldn't be better to do timing conversion in do_versions and not add "Correct Timing" toggle to UI.
Would make UI and code a bit cleaner.

Is there any sensible reason to have such toggle?

I can't think of any reason to have the toggle - the option was added by Alberto, presumably at someone else's request. I think we should use the correct behaviour in all cases, since it was completely unusable before I can't think that anyone would have sub-scenes with animation in any position other than at frame 1.

Still, if it really mustn't change the behaviour of old files, having it done in versions and not in the UI is preferable. The checkbox is clutter and could be very confusing if accidentally un-ticked.

I was not aware of the versions stuff, and indeed that looks like a much better option than having a UI option.

btw @Alberto Mardegan (mardy) I wasn't having a go at you for adding the option, apologies if that was rude. Preserving behaviour of existing files makes sense in a lot of cases, but not here in my opinion :)

In D4072#92539, @Olly Funkster (Funkster) wrote:

btw @Alberto Mardegan (mardy) I wasn't having a go at you for adding the option, apologies if that was rude.

It didn't sound rude at all, no worries :-)

Preserving behaviour of existing files makes sense in a lot of cases, but not here in my opinion :)

I think we should always strive for that: you can never know whether some user already worked around the issue in some other means, and if we fix it unconditionally we'll most likely break his files. I think deciding to apply the fix based on the project's version is still the best idea; then of course it might not be worth the effort, but I think we should at least investigate the possibility.