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

Add TextInput event #1144

Merged
merged 4 commits into from
Aug 24, 2022

Conversation

Jowan-Spooner
Copy link
Collaborator

Text Input event

Options:

  • Variable (the variable that the input will be stored to)
  • Text (will by default be displayed above the line edit)
  • Default
  • Placeholder
  • Allow Empty

Is missing an icon

Text input display node

A control node that has three exported node paths to a text label, an input line edit and a confirmation button.

A node of this type along the needed child nodes was added to the DefaultDialogNode scene.

# Text Input event
Options:
- Variable (the variable that the input will be stored to)
- Text (will by default be displayed above the line edit)
- Default
- Placeholder
- Allow Empty

*Is missing an icon*

# Text input display node
A control node that has three exported node paths to a text label, an input line edit and a confirmation button.

A node of this type along the needed child nodes was added to the DefaultDialogNode scene.
@coppolaemilio
Copy link
Collaborator

coppolaemilio commented Aug 24, 2022

Okay, after testing it I think there should be a way to customize what the input says.
I know this makes things a bit more complicated because of localization but I'm sure people will want to customize the "Please enter some text" prompt.

Same with the "Confirm" option. They could be something to add in the body section of the event.

EDIT: The confirm can be editing via the theme or the node you create

@coppolaemilio coppolaemilio merged commit b69ee8e into dialogic-godot:version-2.0 Aug 24, 2022
@Jowan-Spooner Jowan-Spooner deleted the input-event branch December 26, 2023 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants