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

Make grave inscription text prompt readable #2009

Merged
merged 2 commits into from
Oct 21, 2022

Conversation

chaosvolt
Copy link
Member

@chaosvolt chaosvolt commented Oct 12, 2022

Summary

SUMMARY: Bugfixes "Text prompt for inscribing during burial no longer only 3 characters wide"

Purpose of change

Burying coffins had a major problem with the text prompt being effectively unreadable:
image

Describe the solution

  1. Thanks to Scarf, the popup created in iuse::handle_ground_graffiti now places the text entry below the prompt via a newline, giving it more space to make text entered easier to read and edit.

Describe alternatives you've considered

If I could figure out some way to have basically a newline after the text prompt's title and the text entry, so it'd look neater, that'd be even better. I couldn't find any sign of a way to do that with string_input_popup though. Scarf knew how to do that so we have that now. :D

Testing

  1. Compiled and load tested.
  2. Spawned in and rigged up a coffin ready to be buried
  3. Confirmed the prompt gives more actual typing space (image provided by Scarf):

image

Additional context

src/iuse.cpp Outdated Show resolved Hide resolved
Co-authored-by: scarf <greenscarf005@gmail.com>
casswedson pushed a commit to casswedson/Cataclysm-DDA that referenced this pull request Oct 13, 2022
from cataclysmbnteam/Cataclysm-BN#2009

Co-authored-by: scarf005 <scarf005@users.noreply.github.com>
dseguin pushed a commit to CleverRaven/Cataclysm-DDA that referenced this pull request Oct 14, 2022
from cataclysmbnteam/Cataclysm-BN#2009

Co-authored-by: scarf005 <scarf005@users.noreply.github.com>

Co-authored-by: chaosvolt <chaosvolt@users.noreply.github.com>
Co-authored-by: scarf005 <scarf005@users.noreply.github.com>
@Coolthulhu Coolthulhu self-assigned this Oct 15, 2022
@Coolthulhu Coolthulhu merged commit 1e3682a into cataclysmbnteam:upload Oct 21, 2022
@chaosvolt chaosvolt deleted the grave-fix branch October 21, 2022 15:26
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.

3 participants