besnik / text-adventure-game-js-engine

Text Adventure Game Javascript Engine to create games like old-school console text games in Atari/MS-Dos times. Based on new technologies like HTML5, Javascript, React.
26 stars 7 forks source link

Start to cleanup and restructure #2

Closed coreyar closed 3 years ago

coreyar commented 7 years ago

Creating a pull request to get thoughts on the restructuring and cleanup.

I am not as familiar with the namespace pattern this project uses. I am more familiar with using modules to control scope and using webpack to bundle the code but I can understand how the namespace function is working. I will probably dive more into how the game engine works and see how it fits in with the code in src

besnik commented 7 years ago

Hi Corey, thanks for update. I have following suggestions and then question where I need more clarifications.

few questions I have:

coreyar commented 7 years ago

Are you suggesting to replace /src with the engine code in /poc? I think /poc could be renamed to /example so that a user could install the project and have a working example. I have experience in react and react-native and have a very basic prototype build in react in the /poc folder.

I'll update the README.md with information about running the example.

coreyar commented 7 years ago

I pushed up a much more robust reorganization.