imbrn / ace Goto Github PK

Instead of using this.canvas inside a Scene implementation of draw method, it's more appropriate and favors encapsulation to use canvas as an argument of that.

We can see an example in the moving-ball example:

After this API change it should look like:

draw(canvas: Canvas): void { 
   canvas.fillStyle = "black"; 
   canvas.fill(rect({ x: 0, y: 0, width: this.resolution.width, height: this.resolution.height })); 
    
   canvas.fillStyle = "green"; 
   canvas.fill(circle({ x: this.x, y: this.y, radius: 50 })); 
 }

In this way, we can avoid exposing canvas abstraction to the whole scene but just for the drawing step.

Support sounds as assets.

Support some kind of transformation system so that objects of the game can be translated, scaled and rotated.

Implement a mechanism for using Easing functions. It should be considered related to an animation module.

This module probably depends on #22 (Supports transformation)

For this issue, we must consider scaling up/down images as the sprites should fit into the canvas based on the scene resolution. For this reason, maybe it's important to define a way for scaling areas using the coordinates-system module.

Also, at this first version, we shouldn´t worry about offering a lot of algorithms for scaling up/down images, but I think using the Nearest Neighbour Algorithm is enough for now.