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.

Sign up for GitHub

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

Jump to bottom

Tags on messages/item properties use nonsensical labels such as '23 feet creature' #2644

Closed
krbz999 opened this issue Nov 23, 2023 · 2 comments
Closed

Tags on messages/item properties use nonsensical labels such as '23 feet creature' #2644

krbz999 opened this issue Nov 23, 2023 · 2 comments
Assignees
@Fyorl
Labels
bug Functionality which is not working as intended
Milestone
D&D5E 2.4.1

Comments

@krbz999
Copy link
Contributor

krbz999 commented Nov 23, 2023

Given that such a setup can't be made, the "feet" should be removed.
@Fyorl
Copy link
Contributor

Fyorl commented Nov 28, 2023

Do you have a screenshot/some repro steps?

@Fyorl Fyorl added the bug Functionality which is not working as intended label Nov 28, 2023
@Fyorl Fyorl added this to the D&D5E 2.4.1 milestone Nov 28, 2023
@krbz999
Copy link
Contributor Author

krbz999 commented Nov 28, 2023

  • Create item on actor.
  • Edit item and set target to "1 feet line".
  • Change "line" to "creature" (item details now correctly says "1 creature").
  • Expand item on actor sheet, see that it says "1 feet creature".
  • Use item, see that tag in message footer says "1 feet creature".

image

image

@Fyorl Fyorl self-assigned this Nov 28, 2023
@Fyorl Fyorl closed this as completed Nov 28, 2023
Fyorl added a commit that referenced this issue Nov 28, 2023
@Fyorl
[#2644] Fix adding units to Item target label when it didn't make sense.
8287e57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Fyorl Fyorl

Labels
bug Functionality which is not working as intended
Projects
None yet
Milestone
D&D5E 2.4.1
Development

No branches or pull requests
2 participants
@Fyorl @krbz999