Implement selective auto note preferences and a manager GUI to modify them

[nshcat]

Summary

SUMMARY: Interface "Implement selective auto note preferences and a manager GUI to modify them"

Purpose of change

This adresses issue #33483. The game contains a "auto note" feature, which, if enabled, will automatically add notes to the overmap if map extras are encountered. One quality of life problem with this is, that whether a specific map extra type will receive such an auto note is determined by a JSON flag (called autonote), which requires modding to change. But it is common in a single CDDA run to want to change these preferences - i.e. a group of college kids might be interesting in the early game, but not so much in the late game.

Describe the solution

This pull request implements a configuration system which manages auto note preferences on a per-character basis and such allows players to decide for which types of map extras they want to enable auto notes - and for which they don't.

Additionally, a settings manager GUI is implemented, which allows the user to modify the current auto notes configuration state and is placed next to the auto pickup manager in the main menu:

Describe alternatives you've considered

One could say that also allowing these settings to be stored as a global configuration would be sensitive, but as explained in the introduction it has come to my attention that the decision which auto notes to activate seems to be a very fluid decision, which depends on the particular game stage a character is in. That's why I made the decision to keep the setting on a per-character basis.

Additional context

A few implementation details:

• The data is saved as part of the player save, in an auxiliary JSON file with the suffix .ano.json.
• If such data is not yet present, the subsystem collects all loaded map extras that have the JSON flag autonote already set to true and uses these to build an initial configuration. This is done in order to retain the "status quo" behaviour if the user does not change the settings.
• The autonote JSON flag is still a hard disable - map extra types with this flag set to false are completely disregared by the auto note manager GUI and are not displayed to the user. This stops "internal" extra types such as "Nothing" from being displayed.
• If the map extra subsytem wants to create an auto note, it is first checked if the user allowed this for the particular map extra type at hand, and if not, cancels the operation.

Since this is my first major code change, I would appreciate review of this. Thanks!

[Night-Pryanik]

While I like the idea, I feel this is a big spoiler for new players. How about adding the toggle to turn specific map extra on and off only after first encounter of this map extra? So right after the game start the list of map extras will be empty and will be filling through the course of the game.

[nshcat]

I can implement that, thanks for the input :)

[ZhilkinSerg]

I suggest moving ladder auto-notes to new dedicated menu out of options menu too.

[nshcat]

@Night-Pryanik Thank you for your feedback. I implemented exactly what you asked for.

Here is how the GUI looks on a fresh character that has not discovered any extras yet:

Here is how it looks after a few extras have been discovered:

This PR should now be a spoiler-free experience! :D

[AMurkin]

You don't need all these this-> and a round braces in the ternary operator.

[nshcat]

Thanks again, changes applied.

[AMurkin]

Use // NOLINTNEXTLINE(cata-use-named-point-constants) instead of the point_east please.
Ref #33225.

[nshcat]

All resolved, thanks for the reviews people.

[ZhilkinSerg]

I would like to see some improvements later, but this PR was already big, so merging it as it is:

1. Obsolete auto-notes options completely and move them out of settings menu to auto-note manager.
2. On savegame loading parse existing map extras known to character and fill the list of known extras accordingly.

[nshcat]

@ZhilkinSerg Regarding the improvements you stated: Does "obsolete auto-notes options completely" mean that they should also be stored on a per-character basis, or just the removal from the option menu?

[ZhilkinSerg]

Personally I believe they should be character based, but there are can be different approaches.