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

new char popup edit boxes #39467

Merged
merged 4 commits into from
Apr 14, 2020

Conversation

KorGgenT
Copy link
Member

Summary

SUMMARY: Bugfixes "Uses string popup menus for name, age, and height in character creation"

Purpose of change

Fixes #39411
Also changes the randomizer to randomize all values on the page (description)

Describe the solution

uses string_init_popup instead of checking each keystroke for a key and then using that to enter in a name. honestly i think it looks a lot cleaner in code, even if it adds a couple button presses to character creation.
I changed around the location of some of the catacurses windows to make the "press confirm to edit" show below the "name" field

Testing

image

@KorGgenT KorGgenT added <Enhancement / Feature> New features, or enhancements on existing <Bugfix> This is a fix for a bug (or closes open issue) Character / World Generation Issues and enhancements concerning stages of creating a character or a world labels Apr 11, 2020
src/newcharacter.cpp Show resolved Hide resolved
src/newcharacter.cpp Show resolved Hide resolved
data/raw/keybindings.json Outdated Show resolved Hide resolved
Co-Authored-By: Jianxiang Wang (王健翔) <[email protected]>
@ZhilkinSerg ZhilkinSerg merged commit 668e469 into CleverRaven:master Apr 14, 2020
@KorGgenT KorGgenT deleted the new-char-features-and-fixes branch August 13, 2020 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) Character / World Generation Issues and enhancements concerning stages of creating a character or a world <Enhancement / Feature> New features, or enhancements on existing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Character creation name
3 participants