remove isButtonGroupFocused state

set but never read
palantir · Dec 1, 2018 · 283a380 · 283a380
1 parent b99b717
commit 283a380
Showing 1 changed file with 2 additions and 25 deletions.
diff --git a/packages/core/src/components/forms/numericInput.tsx b/packages/core/src/components/forms/numericInput.tsx
@@ -130,7 +130,6 @@ export interface INumericInputProps extends IIntentProps, IProps {
 
 export interface INumericInputState {
     isInputGroupFocused?: boolean;
-    isButtonGroupFocused?: boolean;
     shouldSelectAfterUpdate?: boolean;
     stepMaxPrecision?: number;
     value?: string;
@@ -290,22 +289,8 @@ export class NumericInput extends AbstractPureComponent<HTMLInputProps & INumeri
         const disabled = this.props.disabled || this.props.readOnly;
         return (
             <ButtonGroup className={Classes.FIXED} key="button-group" vertical={true}>
-                <Button
-                    disabled={disabled}
-                    icon="chevron-up"
-                    intent={intent}
-                    onBlur={this.handleButtonBlur}
-                    onFocus={this.handleButtonFocus}
-                    {...this.incrementButtonHandlers}
-                />
-                <Button
-                    disabled={disabled}
-                    icon="chevron-down"
-                    intent={intent}
-                    onBlur={this.handleButtonBlur}
-                    onFocus={this.handleButtonFocus}
-                    {...this.decrementButtonHandlers}
-                />
+                <Button disabled={disabled} icon="chevron-up" intent={intent} {...this.incrementButtonHandlers} />
+                <Button disabled={disabled} icon="chevron-down" intent={intent} {...this.decrementButtonHandlers} />
             </ButtonGroup>
         );
     }
@@ -360,14 +345,6 @@ export class NumericInput extends AbstractPureComponent<HTMLInputProps & INumeri
         this.invokeValueCallback(nextValue, this.props.onButtonClick);
     };
 
-    private handleButtonFocus = () => {
-        this.setState({ isButtonGroupFocused: true });
-    };
-
-    private handleButtonBlur = () => {
-        this.setState({ isButtonGroupFocused: false });
-    };
-
     private startContinuousChange() {
         // The button's onMouseUp event handler doesn't fire if the user
         // releases outside of the button, so we need to watch all the way