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

Revert "fix: Web - Profile - Hovering over the zoom tool does not display a tooltip." #45297

Closed
wants to merge 1 commit into from

Conversation

yuwenmemon
Copy link
Contributor

Reverts #44788

Candidate for causing the crash here: #45162

@yuwenmemon yuwenmemon requested a review from a team as a code owner July 11, 2024 15:37
@melvin-bot melvin-bot bot requested review from cead22 and removed request for a team July 11, 2024 15:37
Copy link

melvin-bot bot commented Jul 11, 2024

@cead22 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]

@Julesssss
Copy link
Contributor

Julesssss commented Jul 11, 2024

Screenshot 2024-07-11 at 16 50 33

I still can reproduce on web

This comment has been minimized.

1 similar comment
Copy link
Contributor

🧪🧪 Use the links below to test this adhoc build on Android, iOS, Desktop, and Web. Happy testing! 🧪🧪

Android 🤖 iOS 🍎
❌ FAILED ❌ https://ad-hoc-expensify-cash.s3.amazonaws.com/ios/45297/index.html
The QR code can't be generated, because the android build failed iOS
Desktop 💻 Web 🕸️
https://ad-hoc-expensify-cash.s3.amazonaws.com/desktop/45297/NewExpensify.dmg https://45297.pr-testing.expensify.com
Desktop Web

👀 View the workflow run that generated this build 👀

@Julesssss
Copy link
Contributor

Unfortunately the crash is being reported for the iOS test build here.

@Julesssss Julesssss closed this Jul 11, 2024
@Julesssss Julesssss deleted the revert-44788-krishna2323/issue/44029 branch July 11, 2024 16:41
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.

2 participants