-
Notifications
You must be signed in to change notification settings - Fork 157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Restructure backgrounds to levels #90
Comments
@niccokunzmann |
@SilentFlame Sure, you are welcome. You do not have to do everything at once. You can split it up into several pull-requests. |
i am interested in working on this. I have a doubt . In order to add this we need to change some part of code in otherfiles as well right. like loading backgrounds and stuff |
Yes. It is a refactoring that touches a lot. We will gain in the long run. You can start it by creating the object with the interface mentioned above. The methods can be filled step by step and it does not need to be you who does everything. Some reading to the why: https://en.wikipedia.org/wiki/Technical_debt |
@niccokunzmann These levels corresponds to game levels right ? Also
What does they do? |
In the beginning, we can choose levels: Gotham or desert. Show and hide allow to show these levels. start() and stop() are used for playing them. |
Now that I read your code, I know more what I am aiming to. Currently, the data (not functionality) for a level is spread across these files:
If we can put this all together in one file, we can add new levels more easily.
|
In future, if it is possible to add a new background to the svg file and then edit this specification and that is all, this is a huge relief and gives us a lot of freedom in the functionality. |
@niccokunzmann I made a change to the level specification thing to make it look some what clear. Please Have a look and comment on the status. |
@harry-7 I commented. I hope my absence does not discourage you. I was working on this: https://www.youtube.com/watch?v=38JN2JdNnMY&feature=youtu.be |
@niccokunzmann Yeah never mind. The video was cool :). Even though I did not understand it completely. It was cool. |
@niccokunzmann In the above checklist I think we need to couple of changes.
This is not needed anymore . And except level.start() and level.stop() all the remaining are implemented may be you can check them. What do you say ? |
I will check them as soon as they are used in the program and replaced the old functionality. This is as soon as the functionality is not in these files any more. |
hey @niccokunzmann . Can I take this up ? Or is the GCI thing still going on ? |
@harry-7 sure you can take this up, regardless of GCI going on or not. |
@niccokunzmann what is the idea behind level.start() and level.stop() methods? as in what work needs to be done by them? |
Each level has
In a level.js file:
Add methods:
Add functions:
The idea is to create good code. This is the draft and it may need to change once started.
Comment if you would like to work on this.
The text was updated successfully, but these errors were encountered: