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

Use choose_adjacent_highlight in creature container code #38317

Merged
merged 2 commits into from
Apr 2, 2020

Conversation

kevingranade
Copy link
Member

Summary

SUMMARY: None

Purpose of change

Make targeting for using animal containers slightly clearer.

Describe the solution

Highlight valid targets, and adjust the prompt text to hopefully make it clearer that the direction being asked for is originating from the player's position.

Testing

Not done yet, need to verify that it still works correctly and that the highlighting is accurate.

Additional context

Marking draft since this isn't worth mangling strings before a release.

@kevingranade kevingranade added 0.E Feature Freeze Info / User Interface Game - player communication, menus, etc. labels Feb 25, 2020
@kevingranade kevingranade changed the base branch from master to dev March 29, 2020 00:35
@kevingranade kevingranade marked this pull request as ready for review March 29, 2020 00:35
@ZhilkinSerg ZhilkinSerg force-pushed the dev branch 3 times, most recently from ad63e77 to 8e68539 Compare April 1, 2020 12:18
@kevingranade kevingranade force-pushed the dev branch 4 times, most recently from 621a68e to b7106d0 Compare April 2, 2020 07:42
@ZhilkinSerg ZhilkinSerg force-pushed the dev branch 3 times, most recently from 0f30a43 to d432807 Compare April 2, 2020 12:55
@ZhilkinSerg ZhilkinSerg changed the base branch from dev to master April 2, 2020 20:00
@ZhilkinSerg ZhilkinSerg merged commit 85ca8e6 into master Apr 2, 2020
@kevingranade kevingranade deleted the kevingranade-highlight-capturable-creature branch April 9, 2020 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Info / User Interface Game - player communication, menus, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants