Skip to content

Clickable component#4018

Open
m-bert wants to merge 48 commits intomainfrom
@mbert/clickable
Open

Clickable component#4018
m-bert wants to merge 48 commits intomainfrom
@mbert/clickable

Conversation

@m-bert
Copy link
Contributor

@m-bert m-bert commented Mar 9, 2026

Description

This PR introduces new Clickable component, which is meant to be a replacement for buttons.

Note

Docs for Clickable will be added in #4022, as I don't want to release them right away after merging this PR.

borderless

For now, borderless doesn't work. I've tested clickable with some changes that allow borderless ripple to be visible, however we don't want to introduce them here because it would break other things. Also it should be generl fix, not in the PR with new component.

Stress test

Render list with 2000 buttons 50 times (50ms delay between renders), drop 5 best and 5 worst results. Then calculate average.

On android tests were limited to 25 repetitions with dropout of 3 best and worst results. This is because of OOM error.

Stress test example is available in this PR.

Android

$t_{Button}$ $t_{Clickable}$ $\Delta{t}$
BaseButton 1112.44 1194.20 +81.76
RectButton 1517.90 1718.42 +200.52
BorderlessButton 1362.34 1418.16 +55.82

iOS

$t_{Button}$ $t_{Clickable}$ $\Delta{t}$
BaseButton 1114.27 1100.80 -13.47
RectButton 1463.40 1612.50 +149.10
BorderlessButton 1206.60 1272.34 +65.74

Web

$t_{Button}$ $t_{Clickable}$ $\Delta{t}$
BaseButton 53.67 57.99 +4.32
RectButton 93.43 102.40 +8.97
BorderlessButton 99.80 75.06 -24.74

Test plan

New examples

Copilot AI review requested due to automatic review settings March 9, 2026 16:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces a new Clickable component in the react-native-gesture-handler library, intended to serve as a unified replacement for BaseButton, RectButton, and BorderlessButton. The component supports configurable visual feedback (underlay or whole-component opacity changes) and native Android ripple effects. Supporting changes include extracting visual style properties (background color, border radius, etc.) through to the native button in GestureHandlerButton, type improvements to RawButton and BorderlessButton, and a new example screen.

Changes:

  • New Clickable component with configurable animation feedback (feedbackTarget, feedbackType, activeOpacity, underlayColor) and long-press support
  • GestureHandlerButton now passes visual style properties (backgroundColor, borderRadius, border styles) to the native ButtonComponent, and RawButton gets proper type generics
  • Example screen demonstrating Clickable in various configurations (base, rect-like, borderless-like, custom)

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/v3/components/Clickable.tsx New Clickable component with animation, long-press, and ripple support
src/v3/components/index.ts Exports Clickable and ClickableProps (from incorrect path)
src/v3/index.ts Re-exports Clickable from components
src/components/GestureHandlerButton.tsx Extracts visual properties (backgroundColor, border*) into buttonStyle passed to native button
src/v3/components/GestureButtons.tsx Adds type generics to RawButton, extracts ref in BorderlessButton
src/v3/types/NativeWrapperType.ts Adds `
apps/common-app/src/new_api/components/clickable/index.tsx Example screen showcasing Clickable configurations

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@m-bert m-bert marked this pull request as ready for review March 13, 2026 11:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 11 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


export const RawButton = createNativeWrapper(GestureHandlerButton, {
export const RawButton = createNativeWrapper<
ReturnType<typeof GestureHandlerButton>,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it makes sense to change this to unknown (cc @j-piasecki).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No it doesn't, but the comment itself is valid. Why ReturnType<typeof GestureHandlerButton> instead of the actual ref type?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, it might be confusing, but TRef is what is actually wrapped with ref, not ref itself.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, but ReturnType of GestureHandlerButton will be something like ReactNode, no?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As far as I can see it is resolved to JSX.Element

image

Comment on lines +125 to +130
onPressOut?.(e);

if (longPressTimeout.current !== undefined) {
clearTimeout(longPressTimeout.current);
longPressTimeout.current = undefined;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have mixed feelings. I don't like adding more hooks to track onPressIn. Maybe we could call onPressOut after checking e.pointerInside. WDYT @j-piasecki?

Copy link
Member

@j-piasecki j-piasecki Mar 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't pointerInside change triggering onUpdate? If so, then yes.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On Android onUpdate is triggered on each move. On iOS it seems that something is blocking it from being sent as in Android (I'm getting it kinda randomly while panning).

Do we wait with Clickable until we resolve this, or do we merge (and possibly ship) this component while looking for a solution?

KINDA_BLUE: '#5f97c8',
ANDROID: '#34a853',
WEB: '#1067c4',
};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use the common colours, if needed add new colours there.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants