Skip to content
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

re-architecture: Coordinator.startScene(), StoryAssembler.continueScene() should be abstracted #26

Open
Logodaedalus opened this issue Aug 3, 2019 · 0 comments
Labels
enhancement New feature or request UI UI hooks and sample themes

Comments

@Logodaedalus
Copy link
Contributor

We need to further separate (MVC-style) the module dependencies so that all the stuff specific to our displays is done independently of the data structures of Coordinator, etc.

Ideally, it should probably go like:

  1. Coordinator.init()
  2. do stuff with data properties now set in Coordinator for scene title screens
  3. Coordinator.startScene(_Coordinator, playGameScenes[0], true); is called
  4. Some kind of JS object is returned from that
  5. kick off stuff with that, keying off StoryAssembler.handleChoiceSelection() ????

Use cases we need to handle:

  1. Our two display templates retaining all functionality
  2. It being as painless as possible to create new templates (piping in choices, etc)
  3. It being as painless as possible to run StoryAssembler headless, to facilitate the testing harness, data viz, and any other shenanigans.
@Logodaedalus Logodaedalus added enhancement New feature or request UI UI hooks and sample themes labels Aug 3, 2019
@Logodaedalus Logodaedalus changed the title re-architecture: StoryAssembler.continueScene() should just return JS object re-architecture: Coordinator.startScene(), StoryAssembler.continueScene() should be abstracted Aug 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request UI UI hooks and sample themes
Projects
0.2.0 Release
  
Awaiting triage
Development

No branches or pull requests

1 participant