ngctl: small improvements

[wey-gu]

First, it's awesome to have ngctl, really love it, thank you so much!!

• In initial connection, the first line of +--------------------+ should be better newline-ed:

$ngctl console
(root@nebula) [(none)]> show spaces
(root@nebula) [(none)]> +--------------------+
| Name               |
+--------------------+
| "basketballplayer" |
| "shareholding"     |
+--------------------+

• Possible to support UP-btn for history commands or even Ctrl-R?

[veezhang]

@wey-gu

I don't have this problem in first improvement here.

I found that nebula-console is not supported UP-btn or Ctrl-R, is that right?

[wey-gu]

@wey-gu

I don't have this problem in first improvement here.

I found that nebula-console is not supported UP-btn or Ctrl-R, is that right?

Thanks @veezhang !
Yes, we could ignore the first one, maybe to support UP-btn or Ctrl-R would make it more handy to users ;)

[veezhang]

@wey-gu
But I found that the nebula-console have not supported yet. And this feature is depends on it.

How about advise the nebula-console to support first?

[wey-gu]

But I found that the nebula-console have not supported yet. And this feature is depends on it.

How about advise the nebula-console to support first?

Got it, will do so.

[veezhang]

Thanks

[wey-gu]

• ctrl-r for history search

  • It seems console is leveraging go-prompt, which is not supporting history search, while, there is a PR here: Add interactive history search. Reject unprintable characters. c-bata/go-prompt#225 , and it looks promising!
  • Update: I found ctrl-r to search history in nebula-console is already supported :-D

• for up-btn down-btn history

  • both console and go-prompt had been supporting it (ref: https://github.com/c-bata/go-prompt#history ), something is overridden during ngctl importing console as lib here? or?