feat: updated dino walls and bottom wall (#203)

* feat: updated dino walls with coloured assets and fixed shapes and tests

* refactor: priorities for top wall

* refactor: added bottom wall coloured asset and fixed position and priority with dino bottom wall

* chore: added TODO comments for priority future refactors

* test: golden tests for dino and bottom walls

* refactor: bottom boundary render body

* fix: moved up boundary bottom to avoid line separation between dino bottom wall

packages/pinball_components/lib/gen/assets.gen.dart

@@ -57,6 +57,8 @@ class $AssetsImagesBackboardGen {
   /// File path: assets/images/backboard/backboard_scores.png
   AssetGenImage get backboardScores =>
       const AssetGenImage('assets/images/backboard/backboard_scores.png');
+
+  /// File path: assets/images/backboard/display.png
   AssetGenImage get display =>
       const AssetGenImage('assets/images/backboard/display.png');
 }
@@ -309,8 +311,11 @@ class $AssetsImagesSparkyBumperGen {
 class $AssetsImagesSparkyComputerGen {
   const $AssetsImagesSparkyComputerGen();
 
+  /// File path: assets/images/sparky/computer/base.png
   AssetGenImage get base =>
       const AssetGenImage('assets/images/sparky/computer/base.png');
+
+  /// File path: assets/images/sparky/computer/top.png
   AssetGenImage get top =>
       const AssetGenImage('assets/images/sparky/computer/top.png');
 }
@@ -354,8 +359,11 @@ class $AssetsImagesDashBumperMainGen {
 class $AssetsImagesSparkyBumperAGen {
   const $AssetsImagesSparkyBumperAGen();
 
+  /// File path: assets/images/sparky/bumper/a/active.png
   AssetGenImage get active =>
       const AssetGenImage('assets/images/sparky/bumper/a/active.png');
+
+  /// File path: assets/images/sparky/bumper/a/inactive.png
   AssetGenImage get inactive =>
       const AssetGenImage('assets/images/sparky/bumper/a/inactive.png');
 }
@@ -363,8 +371,11 @@ class $AssetsImagesSparkyBumperAGen {
 class $AssetsImagesSparkyBumperBGen {
   const $AssetsImagesSparkyBumperBGen();
 
+  /// File path: assets/images/sparky/bumper/b/active.png
   AssetGenImage get active =>
       const AssetGenImage('assets/images/sparky/bumper/b/active.png');
+
+  /// File path: assets/images/sparky/bumper/b/inactive.png
   AssetGenImage get inactive =>
       const AssetGenImage('assets/images/sparky/bumper/b/inactive.png');
 }
@@ -372,8 +383,11 @@ class $AssetsImagesSparkyBumperBGen {
 class $AssetsImagesSparkyBumperCGen {
   const $AssetsImagesSparkyBumperCGen();
 
+  /// File path: assets/images/sparky/bumper/c/active.png
   AssetGenImage get active =>
       const AssetGenImage('assets/images/sparky/bumper/c/active.png');
+
+  /// File path: assets/images/sparky/bumper/c/inactive.png
   AssetGenImage get inactive =>
       const AssetGenImage('assets/images/sparky/bumper/c/inactive.png');
 }

packages/pinball_components/lib/src/components/boundaries.dart

@@ -26,7 +26,8 @@ class _BottomBoundary extends BodyComponent with InitialPosition {
   /// {@macro bottom_boundary}
   _BottomBoundary()
       : super(
-          priority: 1,
+          // TODO(ruimiguel): set final priority when RenderPriority PR merged.
+          priority: Ball.boardPriority + 2,
           children: [_BottomBoundarySpriteComponent()],
         ) {
     renderBody = false;
@@ -78,7 +79,7 @@ class _BottomBoundarySpriteComponent extends SpriteComponent with HasGameRef {
     this.sprite = sprite;
     size = sprite.originalSize / 10;
     anchor = Anchor.center;
-    position = Vector2(-5.4, 55.8);
+    position = Vector2(-5.4, 55.6);
   }
 }
 

packages/pinball_components/lib/src/components/dino_walls.dart

@@ -31,7 +31,8 @@ class _DinoTopWall extends BodyComponent with InitialPosition {
   ///{@macro dino_top_wall}
   _DinoTopWall()
       : super(
-          priority: 1,
+          // TODO(ruimiguel): set final priority when RenderPriority PR merged.
+          priority: Ball.boardPriority + 1,
           children: [_DinoTopWallSpriteComponent()],
         ) {
     renderBody = false;
@@ -42,7 +43,7 @@ class _DinoTopWall extends BodyComponent with InitialPosition {
 
     final topStraightShape = EdgeShape()
       ..set(
-        Vector2(28.4, -35.1),
+        Vector2(28.65, -35.1),
         Vector2(29.5, -35.1),
       );
     final topStraightFixtureDef = FixtureDef(topStraightShape);
@@ -69,8 +70,8 @@ class _DinoTopWall extends BodyComponent with InitialPosition {
     final bottomCurveShape = BezierCurveShape(
       controlPoints: [
         middleCurveShape.vertices.last,
-        Vector2(21.15, -16),
+        Vector2(21.5, -15.8),
-        Vector2(25.6, -15.2),
+        Vector2(25.8, -14.8),
       ],
     );
     fixturesDef.add(FixtureDef(bottomCurveShape));
@@ -126,6 +127,8 @@ class _DinoBottomWall extends BodyComponent with InitialPosition {
   ///{@macro dino_top_wall}
   _DinoBottomWall()
       : super(
+          // TODO(ruimiguel): set final priority when RenderPriority PR merged.
+          priority: Ball.boardPriority + 1,
           children: [_DinoBottomWallSpriteComponent()],
         ) {
     renderBody = false;
@@ -136,7 +139,7 @@ class _DinoBottomWall extends BodyComponent with InitialPosition {
     const restitution = 1.0;
 
     final topStraightControlPoints = [
-      Vector2(32.4, -8.3),
+      Vector2(32.4, -8.8),
       Vector2(25, -7.7),
     ];
     final topStraightShape = EdgeShape()
@@ -220,6 +223,6 @@ class _DinoBottomWallSpriteComponent extends SpriteComponent with HasGameRef {
     );
     this.sprite = sprite;
     size = sprite.originalSize / 10;
-    position = Vector2(23.8, -9.5);
+    position = Vector2(23.6, -9.5);
   }
 }