Skip to content

Conversation

@neil-marcellini
Copy link
Contributor

Problem

Browser automation tools (and potentially users) have trouble clicking on certain UI elements because SVG icons inside buttons/pressables intercept clicks. The SVG <path> elements become the click target instead of the parent pressable.

Affected Components

  1. Icon component (src/components/Icon/index.tsx) - renders SVGs without pointerEvents="none"
  2. FloatingActionButton (src/components/FloatingActionButton.tsx) - uses raw <Svg> and <AnimatedPath> without pointer-events handling
  3. Navigation tab bar items that use Icon components

Solution

Added pointerEvents="none" to:

  • Icon component: Both the wrapper View and ImageSVG component in the main return statement, as well as the inline rendering case
  • FloatingActionButton: The Svg component wrapping the AnimatedPath

This ensures clicks pass through the SVG elements to the parent pressable components.

Test Plan

  • Verify clicking on navigation tabs (Home, Inbox, Reports, Workspaces, Account) works correctly
  • Verify clicking on the global create button (green + button) works correctly
  • Verify buttons with icons in modals continue to work
  • Verify browser automation tools can click on these elements reliably

Made with Cursor

…vent SVG click interception

Browser automation tools (and potentially users) have trouble clicking on certain
UI elements because SVG icons inside buttons/pressables intercept clicks. The SVG
path elements become the click target instead of the parent pressable.

This fix adds pointerEvents="none" to:
- Icon component's View wrapper and ImageSVG (both main and inline rendering)
- FloatingActionButton's Svg component

Co-authored-by: Cursor <cursoragent@cursor.com>
@neil-marcellini neil-marcellini marked this pull request as ready for review February 5, 2026 19:30
@neil-marcellini neil-marcellini requested review from a team as code owners February 5, 2026 19:30
@melvin-bot melvin-bot bot requested review from joekaufmanexpensify and youssef-lr and removed request for a team February 5, 2026 19:30
@melvin-bot
Copy link

melvin-bot bot commented Feb 5, 2026

@youssef-lr Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@youssef-lr
Copy link
Contributor

Tests failing @neil-marcellini

@joekaufmanexpensify
Copy link
Contributor

No product review needed

@joekaufmanexpensify joekaufmanexpensify removed their request for review February 6, 2026 17:16
@neil-marcellini
Copy link
Contributor Author

Moving this to a draft, it's quite untested.

@neil-marcellini neil-marcellini marked this pull request as draft February 6, 2026 18:09
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.

3 participants