Skip to content

[Web] Align Tap maxDelay default with native (200 ms) #4557

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Shexter wants to merge 3 commits into
base: main
Choose a base branch
from
Open

[Web] Align Tap maxDelay default with native (200 ms) #4557

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
648719a
[Web] Align Tap maxDelay default with native (200 ms)
Shexter Oct 1, 2026
97e50ec
Remove the TapGestureHandler test file
Shexter Oct 2, 2026
b75a6a6
Update the Tap maxDelay default in the v2 builder comment
Shexter Oct 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/docs-gesture-handler/docs/gestures/use-tap-gesture.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ Maximum time, expressed in milliseconds, that defines how fast a finger must be
maxDelay: number | SharedValue<number>;
```

Maximum time, expressed in milliseconds, that can pass before the next tap — if many taps are required. The default value is 500.
Maximum time, expressed in milliseconds, that can pass before the next tap — if many taps are required. The default value is 200.

### numberOfTaps

Expand Down
2 changes: 1 addition & 1 deletion packages/react-native-gesture-handler/src/handlers/TapGestureHandler.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ export interface TapGestureConfig {

/**
* Maximum time, expressed in milliseconds, that can pass before the next tap
* if many taps are required. The default value is 500.
* if many taps are required. The default value is 200.
*/
maxDelayMs?: number;

Expand Down
2 changes: 1 addition & 1 deletion packages/react-native-gesture-handler/src/handlers/gestures/tapGesture.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ export class TapGesture extends BaseGesture<TapGestureHandlerEventPayload> {

/**
* Maximum time, expressed in milliseconds, that can pass before the next tap — if many taps are required.
* The default value is 500.
* The default value is 200.
* @param delay
*/
maxDelay(delay: number) {
Expand Down
2 changes: 1 addition & 1 deletion packages/react-native-gesture-handler/src/v3/hooks/gestures/tap/TapTypes.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ export type TapGestureExternalConfig = CommonTapGestureConfig & {

/**
* Maximum time, expressed in milliseconds, that can pass before the next tap
* if many taps are required. The default value is 500.
* if many taps are required. The default value is 200.
*/
maxDelay?: number;

Expand Down
2 changes: 1 addition & 1 deletion packages/react-native-gesture-handler/src/web/handlers/TapGestureHandler.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import GestureHandler from './GestureHandler';
import type IGestureHandler from './IGestureHandler';

const DEFAULT_MAX_DURATION_MS = 500;
const DEFAULT_MAX_DELAY_MS = 500;
const DEFAULT_MAX_DELAY_MS = 200;
const DEFAULT_NUMBER_OF_TAPS = 1;
const DEFAULT_MIN_NUMBER_OF_POINTERS = 1;

Expand Down
Loading