Danil Alexeev
Danil Alexeev
> ```gdscript > extends Node > > var test := _get_value() > > func _get_value() -> int: > return 5 > ``` See #58542 and #63438.
> > Confirming a change to a multiline cell is done with Ctrl+Enter instead of Enter. > > Should be the opposite. Fixed. > @KoBeWi this is up to you...
Please add "breaks compat" label.
`AnimationPlayer` is for advanced cases, `AnimatedSprite` is for simple ones, but I still want to be able to adjust the duration of individual frames, and not duplicate them. With this...
> The changes done in this PR are nice, but I wonder if it would be better to merge SpriteFrames and AnimatedTexture since their feature sets are so similar. An...
> Also, adding the duration property to `SpriteFrames` would be okay, but `get_animation_total_duration()` doesn't seem necessary. Deleted because most often the total duration of frames is greater than zero. And...
> There's even API changes that are not documented, not even on the commit's description I tried to document all the API changes and clarify the behavior. If I missed...
> The implementation of `pause()` needs to be careful, because similar discussions already exist for `AnimationPlayer` in #56645. Done. I also believe that `play`/`pause`/`stop` behavior should be consistent across classes.
> There is a weird behavior when dragging a frame with a duration set that resets the duration of that frame. Fixed.
@reduz 1. This is consistent with recent `AnimatedTexture` changes (PR #65188). 2. It was mentioned above that in the next PR we can add a tool for multi-editing frame durations...