From 491dd97c601e7ad35a75958d3e15899ebfe7f650 Mon Sep 17 00:00:00 2001
From: Simao Rodrigues <arodrigues.simao@gmail.com>
Date: Wed, 19 Jan 2022 13:00:44 +0000
Subject: [PATCH 1/2] Better differenting visually when Target and SPF is being
 set on all features

---
 .../features/target-spf-item/component.tsx    | 37 +++++++++++++++----
 1 file changed, 29 insertions(+), 8 deletions(-)

diff --git a/app/components/features/target-spf-item/component.tsx b/app/components/features/target-spf-item/component.tsx
index 51acf4a56d..3c9472f4e1 100644
--- a/app/components/features/target-spf-item/component.tsx
+++ b/app/components/features/target-spf-item/component.tsx
@@ -1,9 +1,11 @@
 import React, { useEffect, useRef, useState } from 'react';
+
 import cx from 'classnames';
+
 import Button from 'components/button';
-import Slider from 'components/forms/slider';
-import Label from 'components/forms/label';
 import Input from 'components/forms/input';
+import Label from 'components/forms/label';
+import Slider from 'components/forms/slider';
 
 import { TargetSPFItemProps, Type } from './types';
 
@@ -23,6 +25,8 @@ export const TargetSPFItem: React.FC<TargetSPFItemProps> = ({
 }: TargetSPFItemProps) => {
   const [targetValue, setTargetValue] = useState((target || defaultTarget) / 100);
   const [FPFValue, setFPFValue] = useState(fpf || defaultFPF);
+  const [inputFPFValue, setInputFPFValue] = useState(String(FPFValue));
+
   const sliderLabelRef = useRef(null);
 
   useEffect(() => {
@@ -33,12 +37,18 @@ export const TargetSPFItem: React.FC<TargetSPFItemProps> = ({
     if (typeof fpf !== 'undefined') setFPFValue(fpf);
   }, [fpf]);
 
+  useEffect(() => {
+    setInputFPFValue(String(FPFValue));
+  }, [FPFValue]);
+
   return (
     <div
       key={id}
       className={cx({
-        'bg-gray-700 text-white text-xs pl-5 py-2 relative border-transparent': true,
+        'text-white text-xs pl-5 py-2 mb-2 relative border-transparent': true,
         [className]: !!className,
+        'bg-gray-700': !isAllTargets,
+        'bg-gray-500 border rounded-lg': isAllTargets,
       })}
     >
       <div
@@ -83,18 +93,29 @@ export const TargetSPFItem: React.FC<TargetSPFItemProps> = ({
             <span>100%</span>
           </div>
         </div>
-        <div className="flex flex-col justify-between w-24 px-4 border-l">
+        <div className="flex flex-col justify-between w-24 px-4 border-l border-gray-500">
           <span>{isAllTargets ? 'ALL SPF' : 'SPF'}</span>
           <div className="w-10 mb-6">
             <Input
-              className="px-0 py-1"
+              className="px-0 py-1 rounded"
               theme="dark"
               mode="dashed"
               type="number"
-              value={FPFValue}
+              value={inputFPFValue}
               onChange={({ target: { value: inputValue } }) => {
-                setFPFValue(Number(inputValue));
-                if (onChangeFPF) onChangeFPF(Number(inputValue));
+                setInputFPFValue(inputValue);
+              }}
+              onBlur={() => {
+                // If user leaves the input empty, we'll revert to the original targetValue
+                if (!inputFPFValue) {
+                  setInputFPFValue(String(FPFValue));
+                  return;
+                }
+                // Prevent changing all targets if user didn't actually change it
+                // (despite clicking on the input)
+                if (FPFValue === Number(inputFPFValue)) return;
+                setFPFValue(Number(inputFPFValue));
+                if (onChangeFPF) onChangeFPF(Number(inputFPFValue));
               }}
             />
           </div>

From 47aa7687dc1f7552c4dd878f6ef091e2020fa05d Mon Sep 17 00:00:00 2001
From: Simao Rodrigues <arodrigues.simao@gmail.com>
Date: Tue, 18 Jan 2022 13:07:02 +0000
Subject: [PATCH 2/2] Inputs to support focus related callbacks

---
 app/components/forms/input/component.tsx | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/app/components/forms/input/component.tsx b/app/components/forms/input/component.tsx
index bc39bdf119..e26e38f757 100644
--- a/app/components/forms/input/component.tsx
+++ b/app/components/forms/input/component.tsx
@@ -1,7 +1,10 @@
 import React, { InputHTMLAttributes } from 'react';
-import Icon from 'components/icon';
+
+import { useFocus } from '@react-aria/interactions';
 import cx from 'classnames';
 
+import Icon from 'components/icon';
+
 const THEME = {
   dark: {
     base:
@@ -43,6 +46,9 @@ export interface InputProps extends InputHTMLAttributes<HTMLInputElement> {
     id: string;
     viewBox: string;
   };
+  onFocus?: () => void;
+  onBlur?: () => void;
+  onFocusChange?: (isFocused: boolean) => void;
 }
 
 export const Input: React.FC<InputProps> = ({
@@ -52,10 +58,19 @@ export const Input: React.FC<InputProps> = ({
   disabled = false,
   icon,
   className,
+  onFocus,
+  onBlur,
+  onFocusChange,
   ...props
 }: InputProps) => {
   const st = disabled ? 'disabled' : status;
 
+  const { focusProps: inputFocusProps } = useFocus({
+    onFocus,
+    onBlur,
+    onFocusChange,
+  });
+
   return (
     <div className="relative">
       {icon && (
@@ -79,6 +94,7 @@ export const Input: React.FC<InputProps> = ({
           'pl-10': icon,
           [className]: !!className,
         })}
+        {...inputFocusProps}
       />
     </div>
   );