2019 auto path practice - #129
Conversation
MangoSwirl
commented
Jul 20, 2026
- add stealing and other stuff ?
- change background and locations
- auto
MangoSwirl
left a comment
There was a problem hiding this comment.
Nice work, Leo! Obviously this is a practice excersise so we don't have to get perfectly close to a publishable implementation but I think it would be helpful to get a bit closer than we are currently. Let me know if there's anything I can do to help.
There was a problem hiding this comment.
additional note: I don't think you meant to include these changes on this branch. not super impactful in this particular case but just to be careful, it's worth double-checking your changes before you commit and what branch you're making a new one off of
| childBuilder: (context, teamColor, isHighlighted) => | ||
| inventory.isEmpty ? Container() : inventory.last.icon(), | ||
| inventory.isEmpty | ||
| ? Container(child: Text("f")) | ||
| : inventory.last.icon(), |
There was a problem hiding this comment.
Historically when we've had games where multiple piece types can be in posession (like 2023) we've had both display in a row instead of just the most recent one.
Assuming the "f" wasn't mean to be left in.
| return inventoryAtTimestamp(timestamp) | ||
| .map((a) => PositionedGamePiece(a, positionAtTimestamp(timestamp))) | ||
| .toList(); |
There was a problem hiding this comment.
The gamePiecePositionsAtTimestamp method is meant to return only PositionedGamePieces on the field, not in a robot, since showing the robot's inventory is already handled in animatedPath (line 385)
| extension GamePieceExtension on GamePiece { | ||
| Widget icon( | ||
| {Color color = const Color.from(alpha: 1, red: 1, green: 1, blue: 1)}) { | ||
| Widget icon({Color color = const Color.fromARGB(255, 244, 163, 0)}) { |
There was a problem hiding this comment.
Random design note: for some reason we've always used white for the game piece icon color, regardless of its actual color irl.