From 6f468c670a7657b5daf38d62f7e44a4553727fd8 Mon Sep 17 00:00:00 2001
From: Erik Sundell <erik.sundell@grafana.com>
Date: Fri, 6 Dec 2019 11:50:45 +0100
Subject: [PATCH] UI: Segment Input change (#20925)

* Add input segment

* Rename story

* Cleanup

* Fix lint error

* More cleanup

* Use measure text util

* Keep value in local state
---
 .../src/components/Segment/SegmentInput.tsx   | 23 ++++++++++++++-----
 1 file changed, 17 insertions(+), 6 deletions(-)

diff --git a/packages/grafana-ui/src/components/Segment/SegmentInput.tsx b/packages/grafana-ui/src/components/Segment/SegmentInput.tsx
index ebc026ff0e1..b046f842c57 100644
--- a/packages/grafana-ui/src/components/Segment/SegmentInput.tsx
+++ b/packages/grafana-ui/src/components/Segment/SegmentInput.tsx
@@ -12,18 +12,21 @@ export interface SegmentInputProps<T> extends SegmentProps<T> {
 const FONT_SIZE = 14;
 
 export function SegmentInput<T>({
-  value,
+  value: initialValue,
   onChange,
   Component,
   className,
 }: React.PropsWithChildren<SegmentInputProps<T>>) {
   const ref = useRef(null);
-  const [inputWidth, setInputWidth] = useState<number>(measureText(value.toString(), FONT_SIZE).width);
+  const [value, setValue] = useState<number | string>(initialValue);
+  const [inputWidth, setInputWidth] = useState<number>(measureText(initialValue.toString(), FONT_SIZE).width);
   const [Label, , expanded, setExpanded] = useExpandableLabel(false);
   useClickAway(ref, () => setExpanded(false));
 
   if (!expanded) {
-    return <Label Component={Component || <a className={cx('gf-form-label', 'query-part', className)}>{value}</a>} />;
+    return (
+      <Label Component={Component || <a className={cx('gf-form-label', 'query-part', className)}>{initialValue}</a>} />
+    );
   }
 
   const inputWidthStyle = css`
@@ -39,10 +42,18 @@ export function SegmentInput<T>({
       onChange={item => {
         const { width } = measureText(item.target.value, FONT_SIZE);
         setInputWidth(width);
-        onChange(item.target.value);
+        setValue(item.target.value);
+      }}
+      onBlur={() => {
+        setExpanded(false);
+        onChange(value);
+      }}
+      onKeyDown={e => {
+        if ([13, 27].includes(e.keyCode)) {
+          setExpanded(false);
+          onChange(value);
+        }
       }}
-      onBlur={() => setExpanded(false)}
-      onKeyDown={e => [13, 27].includes(e.keyCode) && setExpanded(false)}
     />
   );
 }