From da9843613a35e897eb5163766b85a6766f7770ba Mon Sep 17 00:00:00 2001
From: Andrei Dumitrescu <5057797+andreidcm@users.noreply.github.com>
Date: Wed, 16 Sep 2020 15:30:18 +0200
Subject: [PATCH] feat: Close CLI when content is empty and pressing backspace

---
 src/ui.cli/cli.js | 39 +++++++++++++++++----------------------
 1 file changed, 17 insertions(+), 22 deletions(-)

diff --git a/src/ui.cli/cli.js b/src/ui.cli/cli.js
index 0e11876..c437656 100644
--- a/src/ui.cli/cli.js
+++ b/src/ui.cli/cli.js
@@ -13,9 +13,6 @@ const commandUI = ({
 }) => {
   const label = blessed.box({
     parent,
-    style: {
-      bg: "gray",
-    },
     content: "/",
     left,
     top,
@@ -27,9 +24,6 @@ const commandUI = ({
     keys: true,
     vi: true,
     mouse: true,
-    style: {
-      bg: "gray",
-    },
     width,
     height,
     top,
@@ -46,7 +40,11 @@ const commandUI = ({
     }
 
     if (key.full === "backspace") {
-      onChange(value.substr(0, value.length - 1))
+      if (value === "") {
+        onCancel()
+      } else {
+        onChange(value.substr(0, value.length - 1))
+      }
     }
 
     if (key.full === "escape") {
@@ -61,30 +59,27 @@ const commandUI = ({
   return [
     input,
     ({ value, isVisible }) => {
-      if (isVisible) {
-        label.show()
-        input.show()
-      } else {
-        label.hide()
-        input.hide()
-      }
-
       input.setContent(value)
 
       /*
-       * Similar to:
-       *
        * useEffect(() => {
-       *   if (isVisible === true) {}
+       *   ...
        * }, [isVisible])
        */
 
-      if (isVisible !== input._.isVisible && isVisible === true) {
-        input.focus()
+      if (isVisible !== input._.isVisible) {
+        if (isVisible === true) {
+          label.show()
+          input.show()
+          input.focus()
+        } else {
+          label.hide()
+          input.hide()
+        }
       }
 
-      /**
-       * Persist state data
+      /*
+       * Local props, acts like prevProps
        */
 
       input._.value = value