pedia commented 3 years ago

Current bug behaviour

Parent render above its children.

Expected behaviour

Parent should render under its children.

Steps to reproduce

class A extends SpriteComponent {
  Future<void> onLoad() async {
     sprite = Sprite('parent.png');
     addChild(SpriteComponent(sprite: 'child.png'));
  }
}

Then parent.png render above child.png. Is this right? Affected components are: SpriteComponent and SpriteAnimationComponent normally.

Flutter doctor output

[✓] Flutter (Channel master, 1.27.0-2.0.pre.79, on macOS 11.0.1 20B50 darwin-x64, locale en-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.5)
[✓] IntelliJ IDEA Community Edition (version 2019.1.3)
[✓] VS Code (version 1.53.2)
[✓] Connected device (2 available)

• No issues found!

More environment information

[✓] Android :android:
[✓] iOS 🍎
[✓] Web :browser:

flame-version: 1.0-rc6

Now:

class SpriteComponent {
void render(Canvas canvas) {
    super.render(canvas);

    sprite?.render(
      canvas,
      size: size,
      overridePaint: overridePaint,
    );
  }
}

should be:

class SpriteComponent {
void render(Canvas canvas) {
    // render children
    sprite?.render(
      canvas,
      size: size,
      overridePaint: overridePaint,
    );

    // render self
    super.render(canvas);
  }
}