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

Ability to show Token name in speech/thought bubble #2885

Closed
cwisniew opened this issue Aug 13, 2021 · 4 comments · Fixed by #2895
Closed

Ability to show Token name in speech/thought bubble #2885

cwisniew opened this issue Aug 13, 2021 · 4 comments · Fixed by #2895
Assignees
Labels
feature Adding functionality that adds value tested This issue has been QA tested by someone other than the developer.

Comments

@cwisniew
Copy link
Member

Is your feature request related to a problem? Please describe.
Some time in the past I believe -- or have been lead to believe -- the speech and thought bubble used to have the impersonated token name instead of the players name. This is more useful for some people when they want to display these during roleplaying sessions.

Describe the solution you'd like
We should add another token attribute that allows a user to specify a name for the Token that should be displayed for speech/thought bubbles.

  1. If mouse is hovering over a token with this new attribute then initiating the speech/thought bubble should use the name in this attribute (as long as the player is an owner of this token)
  2. If the player has a token impersonated with this attribute and the mouse is not hovering over another token with hovering over a token with this attribute then the value for the impersonated token should be used
  3. If neither of the above is the case the players name should be used for the speech / thought bubble

This will depend on where the mouse is at time the speech / though bubble is initiated and does not change as its moved around.

Question: Should the arrow / finger pointer also use this value or the players name?

Describe alternatives you've considered
Not having the new attribute and defaulting to impersonated name which some people may not want this change from current behaviour.

@cwisniew cwisniew added the feature Adding functionality that adds value label Aug 13, 2021
@cwisniew cwisniew self-assigned this Aug 13, 2021
@Phergus
Copy link
Contributor

Phergus commented Aug 13, 2021

I think someone is imagining things in regards to MT having done this in the past. I checked multiple 1.3 versions back to 1.3.b43 and they all used the player name. 1.2 didn't have impersonation.

It seems like a good idea in any case.

I think of the arrow & finger pointers as being OOC actions. The GM or players pointing out locations/objects/tokens to each other. Maybe for players?

@Azhrei
Copy link
Member

Azhrei commented Aug 13, 2021

How are you going to handle stacked tokens? My party has a mage with a telepathic familiar and this would be a cool feature, but the Tiny familiar occupies the same square as the mage so I don't see how this would be usable for both tokens...

@Phergus Phergus added this to To do in MapTool 1.10.0 via automation Aug 18, 2021
@Phergus Phergus moved this from To do to In progress in MapTool 1.10.0 Aug 18, 2021
@Phergus Phergus moved this from In progress to Review in progress in MapTool 1.10.0 Aug 19, 2021
@Phergus Phergus moved this from Review in progress to Reviewer approved in MapTool 1.10.0 Aug 20, 2021
@Phergus Phergus added the tested This issue has been QA tested by someone other than the developer. label Aug 20, 2021
@Phergus
Copy link
Contributor

Phergus commented Aug 20, 2021

Tested. Player Name or Speech Bubble Name appears in the speech/thought bubbles depending on the situation. See #2895 for details.

@Azhrei
image
image

@Azhrei
Copy link
Member

Azhrei commented Aug 20, 2021

It looks like Craig replied in the PR, not here. But based on the images he attached there, I'd say the approach towards stacked tokens should work. 👍

@Phergus Phergus moved this from Reviewer approved to Done in MapTool 1.10.0 Oct 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding functionality that adds value tested This issue has been QA tested by someone other than the developer.
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

3 participants