textadventures / quest

Create text adventure games
http://textadventures.co.uk/quest
MIT License
304 stars 69 forks source link

"Continue" link #624

Closed alexwarren closed 8 years ago

alexwarren commented 10 years ago

The "Continue" link that shows for a "wait" is currently in the same div as the command box. This means if you hide the command box, you don't see "continue" links any more.

What we should be doing instead is writing "continue" links in-line with game text, then hiding them after use. This can use the new functionality for hiding text sections. Then we wouldn't need a "continue" link in the same location as the command bar, although maybe this will still be required for legacy games.

When this has been implemented, the "wait" command can be re-enabled in the script adder for gamebook mode.

Work Item Details

Original CodePlex Issue: Issue 1250 Status: Proposed Reason Closed: Unassigned Assigned to: Unassigned Reported on: Feb 28, 2013 at 9:33 AM Reported by: alexwarren Updated on: Nov 28, 2013 at 11:11 AM Updated by: alexwarren

alexwarren commented 10 years ago

On 2013-11-28 19:11:34 UTC, alexwarren commented:

It would make sense to try to implement a pure ASL version of the "wait" command. This would need to be a function that takes callback, similar to ShowMenu.

To implement this, we would need to implement #1294 at the same time to prevent "on ready" scripts from running while the ASL wait was in effect.

In implementing #1294, it would be good to capture the script context so that variables can be referenced inside the wait script. This would be needed for the new wait function to act like the old script command.

I'm thinking that in both ShowMenu and the new Wait function, we'd need to call new script commands to capture the context and push a callback onto the list. Then in ShowMenuReponse and a new EndWait function, we'd need a new version of the "invoke" script command to pop the callback off the list and run it using the captured context.

alexwarren commented 8 years ago

Not planning any changes here