Prompt resource doesn't remove cursor when ENTER key is pressed.

[richardmcsong]

Code:

prompt = promptui.Prompt{
	Label: "Prompt 3",
}
_, err = prompt.Run()

The string that I get back from prompt.Run doesn't include the cursor, though.

[chriszzt]

You could try my fork. I fixed it in some way.
Since it is a bit hacky, I might not create a pr for now.
https://github.com/tigergraph/promptui

[laverya]

Is the issue just that the length in bytes is different than the length in runes then @chriszzt? Assuming https://github.com/tigergraph/promptui/pull/2/files is the relevant PR and not https://github.com/tigergraph/promptui/pull/1/files, I mean - since 1 was much larger than 2

[chriszzt]

https://github.com/tigergraph/promptui/pull/2/files

Oh, actually c.input is of type []rune, that is why I deleted unnecessary type conversion in my pr 2.

By the way, the cursor issue is handled by
https://github.com/tigergraph/promptui/blob/master/prompt.go#L236

[jamesdobson]

Created PR #147 which extracts the commit PR #118 by @richardmcsong that deals specifically with this issue (PR #118 also contains additional functionality), hopefully making PR #147 easier to accept.

Also, PR #147 does pass the CI build, which also hopefully makes it easier to accept.