D8H
> > * Animations must be started each time a new one is set because animations may not loop and the ladder state can also stop it (if the character...
I did some changes: * Simplify events * Move the custom animation explanation in the "disable animation" action description. * Remove the description on animation name property. * The default...
I changed the descriptions to: ``` Sway objects like grass in the wind. ``` ``` This extension makes objects sway at different times. It can be used to: - Simulate...
I gave a quick look, it looks good. A max speed = 0 could be a valid use-case to stop the object.
@tristanbob I think that @Alios5 was asking for a full review but it was merged in the community list. Did you do a full review? Should we open another PR...
> I did not perform a full review. I saw the extension was placed in the community folder, so that is how I treated it. > > We could do...
As this PR has been inactive for 1 year, I'm closing it. Feel free to open it back if you want to continue working on this extension.
> * doStepPostEvents condition is inverted. It would be easier to read if the condition was NOT inverted and the direction of the inequality was changed (it seems the TiledUnitsBar...
Please use video instead of gif because gif can't be navigated and have poor quality/size ratio. > * There is two objects called "Resource bar", we should rename the first...
> "Thumb" as a name is very clear, but "Resource" is less. What about "Filling"? > Indeed. Before I rename it, does anyone else have suggestions for the name? >...