Skip to content

Use object's display alias for ShowMenuResponse during disambiguation#1389

Merged
alexwarren merged 3 commits intotextadventures:masterfrom
KVonGit:ShowMenuResponse-uses-displayalias
Jan 20, 2025
Merged

Use object's display alias for ShowMenuResponse during disambiguation#1389
alexwarren merged 3 commits intotextadventures:masterfrom
KVonGit:ShowMenuResponse-uses-displayalias

Conversation

@KVonGit
Copy link
Copy Markdown
Contributor

@KVonGit KVonGit commented Jan 20, 2025

  • Use GetDisplayAlias to print selection made in ShowMenuResponse
  • Add "donotclearmenus" boolean attribute to game object, default false
  • Add "showmenuresponses" boolean attribute to game object, default false
  • Add DisableMenu function
  • Add DisableMenuOutputSection function
  • Add JS disableMenuOutputSection function
  • Add editor controls for new game object attributes

Screenshot (Editor Controls)

404703720-cb2af34b-dbd8-41b3-99c1-600741e899ed


Animated GIF 1

quest-pertex-showmenuresponse-mod


Animated GIF 2

quest-pertex-showmenuresponse-mod-normal-settings


Test game:

Testing GetDisplayAlias in ShowMenuResponse.zip

KVonGit and others added 3 commits January 19, 2025 17:49
- Use `GetDisplayAlias` to print selection made in `ShowMenuResponse`
- Add "donotclearmenus" boolean attribute to game object, default false
- Add "showmenuresponses" boolean attribute to game object, default false
- Add `DisableMenu` function
- Add DisableOutputSection function
- Add JS `disableCmdLink` function
- Add editor controls for new game object attributes

Co-Authored-By: Pertex <Pertex@users.noreply.github.com>
Change function names to something more logical
- Check `game.showmenuresponses` regardless of `game.disambiguating` value
@alexwarren alexwarren merged commit bbaab04 into textadventures:master Jan 20, 2025
@KVonGit KVonGit deleted the ShowMenuResponse-uses-displayalias branch January 20, 2025 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants