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

'E' to talk does nothing in conversations #483

Open
StenBone opened this issue Aug 2, 2021 · 7 comments
Open

'E' to talk does nothing in conversations #483

StenBone opened this issue Aug 2, 2021 · 7 comments
Assignees
Labels
bug (important) This needs to be fixed before shipping, but not as a first priority

Comments

@StenBone
Copy link

StenBone commented Aug 2, 2021

Short description
After the player has initiated a conversation with an NPC by pressing the 'E' key, the prompt for the 'E' key remains on screen even though pressing 'E' does nothing to move the conversation forward. The player must press the 'Space' key to move the conversation forward. This leads to confusion for the player just starting the game without knowledge of the key bindings(experienced first-hand).

Expected behaviour
A clear and concise description of what you expected to happen instead.

To Reproduce
Steps to reproduce the behavior:

  1. Open TestingGround scene
  2. Click the Play button
  3. Move the player character over to the rabbit(BardHare (1)) close by the player spawn.
  4. Press 'E' to initiate NPC conversation.
  5. Notice that pressing 'E' again doesn't move the conversation forward, even though the 'E' key prompt is still displayed. The only way to move the conversation forward is by pressing 'Space', which isn't prompted by the UI on screen.

Notes

  • Simple fix. Either we change the key to move conversations forward to the 'E' key, or we change the prompt that remains onscreen to switch from 'E' to 'Space' once a conversation is started.

image

@Haxsen
Copy link

Haxsen commented Aug 4, 2021

I say we should switch to "Space", "E" is a button for 'interact' action usually. Also need to change 'Talk' message into 'Skip' or 'Next'.

(Hi everyone, I am new in this community. I hope ya'll doing great.)

@Haxsen
Copy link

Haxsen commented Aug 5, 2021

How about this?

UOP1dialogueSkip.mp4

@normyp
Copy link

normyp commented Aug 17, 2021

@Haxsen looks good to me. Are you going to submit a PR?

@Haxsen
Copy link

Haxsen commented Aug 17, 2021

@normyp I am not sure yet since there's no response of the developers.

@ciro-unity ciro-unity added the bug (important) This needs to be fixed before shipping, but not as a first priority label Aug 23, 2021
@ciro-unity
Copy link
Contributor

We're taking a look at this. @uChema is doing a lot of work on the UI, so it might not end up being an issue. I've marked it as Important though, so we'll come back to it! No PR needed for now.

Thanks 🙏🏻

@uChema
Copy link
Contributor

uChema commented Aug 23, 2021

Hey everyone !
Thank you for opening this issue. The solution proposed looks good and we were discussing the issue in other systems as well such as Cutscenes.
The problem here is that the interaction prompt "E" is independent from the DialogueSystem and is triggered each time a character is colliding with the player. I'll try to change the "E" button to "Space" as @Haxsen proposed . The GameState system that we implemented lately could help us in detecting if a character is in Dialogue, and change or hide the prompt.

@Haxsen
Copy link

Haxsen commented Aug 25, 2021

Hi @uChema hope you're doing well.
My solution was very simple, i.e. Enabling the skip button overlay whenever a conversation is started. I thought this simple solution will be fine in this case. But I would like to see your solution to this (please notify when resolved).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug (important) This needs to be fixed before shipping, but not as a first priority
Projects
None yet
Development

No branches or pull requests

5 participants