Skip to content

Commit 0ba1839

Browse files
committed
Add explicit return types to fix linting warnings
Add void return types to event handlers and cleanup functions in touch control components
1 parent a41d8f9 commit 0ba1839

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/game/mobile/TouchButtons.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,25 +17,25 @@ export const TouchButtons: React.FC<TouchButtonsProps> = ({
1717
const [firePressed, setFirePressed] = React.useState(false)
1818
const [shieldPressed, setShieldPressed] = React.useState(false)
1919

20-
const handleFireStart = (e: React.TouchEvent | React.MouseEvent) => {
20+
const handleFireStart = (e: React.TouchEvent | React.MouseEvent): void => {
2121
e.preventDefault()
2222
setFirePressed(true)
2323
onControlsChange({ fire: true, shield: shieldPressed })
2424
}
2525

26-
const handleFireEnd = (e: React.TouchEvent | React.MouseEvent) => {
26+
const handleFireEnd = (e: React.TouchEvent | React.MouseEvent): void => {
2727
e.preventDefault()
2828
setFirePressed(false)
2929
onControlsChange({ fire: false, shield: shieldPressed })
3030
}
3131

32-
const handleShieldStart = (e: React.TouchEvent | React.MouseEvent) => {
32+
const handleShieldStart = (e: React.TouchEvent | React.MouseEvent): void => {
3333
e.preventDefault()
3434
setShieldPressed(true)
3535
onControlsChange({ fire: firePressed, shield: true })
3636
}
3737

38-
const handleShieldEnd = (e: React.TouchEvent | React.MouseEvent) => {
38+
const handleShieldEnd = (e: React.TouchEvent | React.MouseEvent): void => {
3939
e.preventDefault()
4040
setShieldPressed(false)
4141
onControlsChange({ fire: firePressed, shield: false })

src/game/mobile/TouchJoystick.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ export const TouchJoystick: React.FC<TouchJoystickProps> = ({
8080
})
8181

8282
// Cleanup
83-
return () => {
83+
return (): void => {
8484
manager.destroy()
8585
managerRef.current = null
8686
}

0 commit comments

Comments
 (0)