Conversation
…to robot-minigame
…to robot-minigame
RichDom2185
left a comment
RichDom2185
left a comment
There was a problem hiding this comment.
Did a quick run through as I'm outside right now, so just some preliminary comments, thanks!
lhw-1
left a comment
lhw-1
left a comment
There was a problem hiding this comment.
More of an overarching question - do you think it makes sense to frame it just as a maze minigame rather than a robot minigame? The idea of having it be a robot seems more context dependent within SA game, but as a module (which is published for not just Source Academy but for Source as a whole), it might be a nice idea to frame it as a general purpose maze minigame.
(Asking because this would necessitate some variable name changes)
| drawBorders(ctx, width, height); | ||
| drawAreas(ctx, areas); | ||
| drawRobot(ctx, robot); |
There was a problem hiding this comment.
Does it make sense to allow some optional parameters (e.g. colors, stroke style)?
| }; | ||
|
|
||
| // The speed to move at | ||
| const ANIMATION_SPEED : number = 2; |
There was a problem hiding this comment.
Might be nice to allow this to change as well
5 participants
Description
Implement the
robot-minigamemodule for use as a Source Academy Game minigame.Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.
Checklist: