Skip to content
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.

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

Fix close tab btn #5598

Merged
merged 5 commits into from
Apr 5, 2024
Merged

Fix close tab btn #5598

merged 5 commits into from
Apr 5, 2024

Conversation

brunobar79
Copy link
Member

Fixes APP-1284

What changed (plus any additional context for devs)

Fixes two issues with the close button.

  • Pointer events animated style were preventing clicking on children
  • TapGesture of the tab colliding with the Button Tap

Screen recordings / screenshots

close.button.fix.mov

What to test

Copy link

linear bot commented Apr 4, 2024

@brunobar79 brunobar79 merged commit f9109ac into develop Apr 5, 2024
6 checks passed
@brunobar79 brunobar79 deleted the @bruno/fix-close-tab-btn branch April 5, 2024 20:00
BrodyHughes added a commit that referenced this pull request Apr 10, 2024
…e-changes

* 'develop' of github.com:rainbow-me/rainbow: (44 commits)
  allow open in new tab (#5610)
  added warning for unknown price impact (#5597)
  fix cloudflare protection (#5609)
  improve type checking on web preferences (#5607)
  fix scrolltoindex firing on last card dismissal (#5606)
  make account network switcher work (#5604)
  Dapp browser fixes (#5596)
  Fix close tab btn (#5598)
  browser: add account context menu (#5603)
  Fix instant screenshot setting (#5602)
  swaps: bump sdk (#5583)
  Browser: tab transitions, state update queue (#5582)
  audit: undici (#5594)
  [SWAPS V2]: Add token search logic and ability to select assets (#5547)
  swaps v2 gas (#5526)
  tx sim: other natives (#5585)
  Brody/bump 1.9.21 3 (#5588)
  fix sheet bg (#5590)
  Only hold the active tab ref in BrowserContext (#5579)
  Dapp browser: disable tab closing for empty state (#5573)
  ...
BrodyHughes added a commit that referenced this pull request Apr 10, 2024
* 'develop' of github.com:rainbow-me/rainbow:
  make account network switcher work (#5604)
  Dapp browser fixes (#5596)
  Fix close tab btn (#5598)
  browser: add account context menu (#5603)
  Fix instant screenshot setting (#5602)
  swaps: bump sdk (#5583)
  Browser: tab transitions, state update queue (#5582)
  audit: undici (#5594)
  [SWAPS V2]: Add token search logic and ability to select assets (#5547)
  swaps v2 gas (#5526)
  tx sim: other natives (#5585)
  Brody/bump 1.9.21 3 (#5588)
  fix sheet bg (#5590)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants