robotopia-x / robotopia

:robot: Introducing kids to coding with tiny virtual robots!
https://robotopia-x.github.io/
MIT License
511 stars 56 forks source link

Ending Resources #96

Closed timgrossmann closed 7 years ago

timgrossmann commented 7 years ago

Please merge #87 First

Part of Issue #82

I'm not really sure if a separate component fits this case, but I thought the health component is better for actual "active" elements like robots/towers.

What do you think?

Edit: That is what i meant:
screen shot 2017-02-20 at 20 27 53

The collectable components now get removed partially until fully removed

Code review is really needed here, right now the renderer is separate and probably could be integrated better

Update:

There are now 3 "types" or resources:

You can set the values of the gems aka. resources with different "chunks" e.g. RED_GEM with 25/chunk and BLUE_GEM with 10/chunk. The robots also carry the same color of resource with them instead of just blue

paulsonnentag commented 7 years ago

Just a quick note for the future. If you create a pull request which is forked from another branch which hasn't been merged can you reference that branch so the pull requests can be easily reviewed in order