From ae829383141118e9d99fcf48481c64682f9ac633 Mon Sep 17 00:00:00 2001 From: Jordan Eldredge Date: Fri, 15 Feb 2019 07:32:14 -0800 Subject: [PATCH] Fix some react typings --- js/components/ClickedDiv.tsx | 11 +---------- js/components/MainWindow/Marquee.tsx | 2 +- js/components/ResizeTarget.tsx | 2 +- 3 files changed, 3 insertions(+), 12 deletions(-) diff --git a/js/components/ClickedDiv.tsx b/js/components/ClickedDiv.tsx index 7b6bc8cf..f928a6a6 100644 --- a/js/components/ClickedDiv.tsx +++ b/js/components/ClickedDiv.tsx @@ -1,20 +1,11 @@ import React, { useState } from "react"; import classnames from "classnames"; -type DivProps = React.DetailedHTMLProps< - React.HTMLAttributes, - HTMLDivElement ->; - -interface Props extends DivProps { +interface Props extends React.HTMLAttributes { className?: string; onMouseDown?: (e: React.MouseEvent) => void; } -interface State { - clicked: boolean; -} - // Winamp has a strange behavior for the buttons at the top of the main window. // It shows through to the main background sprite until the first time that it's // clicked, and then it shows the dedicated undepressed sprite thereafter. diff --git a/js/components/MainWindow/Marquee.tsx b/js/components/MainWindow/Marquee.tsx index 3c7a292b..65897e49 100644 --- a/js/components/MainWindow/Marquee.tsx +++ b/js/components/MainWindow/Marquee.tsx @@ -79,7 +79,7 @@ class Marquee extends React.Component { } } - handleMouseDown = (e: React.MouseEvent) => { + handleMouseDown = (e: React.MouseEvent): void => { const xStart = e.clientX; this.setState({ stepping: false }); const handleMouseMove = (ee: MouseEvent) => { diff --git a/js/components/ResizeTarget.tsx b/js/components/ResizeTarget.tsx index 6c6f61f3..257cc79e 100644 --- a/js/components/ResizeTarget.tsx +++ b/js/components/ResizeTarget.tsx @@ -14,7 +14,7 @@ interface Props { } export default class ResizeTarget extends React.Component { - handleMouseDown = (e: React.MouseEvent) => { + handleMouseDown = (e: React.MouseEvent) => { // Prevent dragging from highlighting text. e.preventDefault(); const [width, height] = this.props.currentSize;