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

Ability to configure user-defined map extra symbols #48794

Merged

Conversation

AMurkin
Copy link
Contributor

@AMurkin AMurkin commented May 8, 2021

Summary

Features "Implemented user defined map extra symbols"

Purpose of change

Continue #39430.
Provide a basic GUI for customizing user-defined map extra symbols requested in #38648.

Describe the solution

Map a hotkey to CHANGE_MAPEXTRA_CHARACTER action.
Add data structures and methods to manipulate user-defined map extra symbols.
Saving and loading of user-defined map extra symbols.
Implement basic UI.

Describe alternatives you've considered

Not allowing user to set custom symbols and remove unused action CHANGE_MAPEXTRA_CHARACTER.
Experimental

Testing

Remap several symbols in Auto note manager.
Save changes.
Newly discovered map extras got new symbols.
Custom symbols correctly loaded after restart.

Additional context

Non-ASCII symbols are still unsupported.
Please check new strings for any errors.

Screenshots:

  • Manager
    0

  • Symbol prompt
    1

  • Color picker
    2

  • Overmap with user-defined map extra symbols
    overmap

  • Disabled auto notes (map extras) state
    4

  • On exit prompt
    5

AMurkin added 3 commits May 8, 2021 16:15
But prompt the user on exit if all auto notes are disabled
to enable them, when auto notes (map extras) are enabled.
UI doesn't fit "(map extras)" clarification.
In any case, at the moment only the map extras auto notes
are configured in this window.
@Maleclypse Maleclypse added Info / User Interface Game - player communication, menus, etc. Map / Mapgen Overmap, Mapgen, Map extras, Map display Quality of Life QoL: degree to which players are comfortable, and able to enjoy CDDA labels May 9, 2021
Copy link
Member

@kevingranade kevingranade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but no time to test at the moment, all I'm looking for before merging is a simple "poke at the menu and confirm it seems to work as intended" session.

@kevingranade kevingranade merged commit cd85eb8 into CleverRaven:master Jul 10, 2021
@actual-nh actual-nh added 0.F Backport Candidate <Enhancement / Feature> New features, or enhancements on existing and removed 0.F Feature Freeze labels Jul 22, 2021
ZhilkinSerg added a commit to ZhilkinSerg/Cataclysm-DDA that referenced this pull request Aug 12, 2021
@AMurkin AMurkin deleted the user-defined-map-extra-symbols branch September 7, 2022 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Enhancement / Feature> New features, or enhancements on existing Info / User Interface Game - player communication, menus, etc. Map / Mapgen Overmap, Mapgen, Map extras, Map display Quality of Life QoL: degree to which players are comfortable, and able to enjoy CDDA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants