-
Notifications
You must be signed in to change notification settings - Fork 109
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
Modular app.js #18
Comments
Sounds cool ! Try to work on it , if you want any help feel free to contact. |
Yeah I was going to suggest the same. The way the game is made right now is to check for CSS classes for any game logic, which is not ideal. It should definitely be separated into MVC components. |
@RishabhDevbanshi Can I work on this? I think adding more levels will be great too. |
Sure ! |
All the code is currently in app.js
I think separating the code in different files and make it modular will make project more structured.
MVC separates an application into three components - Model, View, and Controller.
This will make it easier for collaborators to work together and make merging requests easier by reducing conflicts.
Adding new updates and features will also be easier in future.
The text was updated successfully, but these errors were encountered: