This repository has been archived by the owner on Sep 12, 2021. It is now read-only.
Added Recipient to PDA Message Input Box Title #511
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
About The Pull Request
I passed the PDA list to msg_input() to pass the target's name to the title in stripped_input(). If there's more than one recipient in the target list, "Multiple PDAs" is displayed as the recipient. If both names together are too long, the title gets cut off. I'm ignoring this, since you can see enough to tell who you're messaging, assuming the sender hasn't decided to make their name/job combo excessively long.
One recipient example:
Multiple recipients example:
Title is too long example:
Why It's Good For The Game
Makes it less likely for your messages about mutiny against the captain, to get sent to the captain.
Changelog
馃啈
tweak: PDA send-message box says who the message is being sent to
/:cl: