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

[Feature] MAPINFO flags for various automap kill/item/secret customization #2926

Open
MajorCooke opened this issue Feb 4, 2025 · 2 comments

Comments

@MajorCooke
Copy link
Contributor

MajorCooke commented Feb 4, 2025

Simple flags that change how the kill/items/secrets in the automap behave. Assume <Name> is a shoe-in for kills, items and secrets to keep things simple.

  • Hide<Name>Cap: The automap won't print out the totals portion, allowing for a map to seemingly have unknowable enemies remaining. Automatically implied by the following flag.
  • Count<Name>Down: Inverts the automap counters to display how many are remaining instead of accrued.
  • Hide<Name>s: The automap won't print this specific statistic or its name at all.

These would be handy for exploration maps where there are no kills, items and/or secrets to find, or leaving it more of a mystery, and to de-emphasize 100% completing wherever it's not necessary, for example.

@madame-rachelle
Copy link
Collaborator

madame-rachelle commented Feb 4, 2025

After some point I think you have to start leveraging ZScript to its full potential.

This seems to introduce complexity where it's likely most of the code can be directly in a mod instead. ZScript already allows you to change automap counts for items, kills, and secrets. If you need custom goals they can already be printed while the map is active and tallied in the intermission, as well.

Unless there's something that clearly tells me there's a massive benefit for the engine as a whole, my opinion is no on this, sorry. This is too much complexity for a niche feature that can already be handled with scripts.

EDIT: I would be in support for Hide<Name>s though, for the reasons above.

@MajorCooke
Copy link
Contributor Author

If there's an overridable function or something that can alter the display count in the automap which I'm not aware of, then sure! I'm just not sure what it is.

Otherwise, I would like to recommend a set of virtual functions somewhere that'd allow for changing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants