Skip to content

fix(iOS): remove deprecated method from RCTPerfMonitor & fix position#43058

Closed
krozniata wants to merge 2 commits into
facebook:mainfrom
krozniata:fix/rct-perf-monitor
Closed

fix(iOS): remove deprecated method from RCTPerfMonitor & fix position#43058
krozniata wants to merge 2 commits into
facebook:mainfrom
krozniata:fix/rct-perf-monitor

Conversation

@krozniata
Copy link
Copy Markdown
Contributor

@krozniata krozniata commented Feb 16, 2024

Summary:

This PR removes usage of deprecated statusBarFrame method in RCTPerfMonitor . Instead RCTPerfMonitor now uses safeAreaInsets which also fixes issue causing Perf Monitor to appear under corner in landscape mode on e.g. iPhone 15 Pro. It also fixes initial position of expanded state which was causing it to render under notch.

Also removed duplicate background color setting

Changelog:

[IOS] [REMOVED] - Remove usage of deprecated statusBarFrame method
[IOS] [FIXED] - Fix position of RCTPerfMonitor in landscape mode & expanded mode

Test Plan:

RNTester builds and runs successfully, RCTPerfMonitor works and displays correctly

@krozniata krozniata changed the title feat(iOS): remove deprecated method from RCTPerfMonitor & fix position fix(iOS): remove deprecated method from RCTPerfMonitor & fix position Feb 16, 2024
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Feb 16, 2024
@krozniata
Copy link
Copy Markdown
Contributor Author

@cipolleschi Could you take a look? 👀

Copy link
Copy Markdown
Contributor

@cipolleschi cipolleschi left a comment

Choose a reason for hiding this comment

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

Hi @krozniata, thank you so much for this contribution and for your patience!

@cipolleschi
Copy link
Copy Markdown
Contributor

Could you perhaps rebase and fix the conflict? 🙏

@krozniata krozniata force-pushed the fix/rct-perf-monitor branch from 70196c6 to a585862 Compare June 25, 2024 08:53
@krozniata
Copy link
Copy Markdown
Contributor Author

@cipolleschi done 👌

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@cipolleschi
Copy link
Copy Markdown
Contributor

Hi there! To set expectations, there are some internal tests that are failing, and I need to understand what's going on.
I don't think I'll have time today, and next week I'll be on PTO, so this has to wait until I'm back.

I hope that this won't cause any issue!

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@cipolleschi merged this pull request in 258f41a.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Jul 8, 2024
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 8, 2024

This pull request was successfully merged by @krozniata in 258f41a.

When will my fix make it into a release? | How to file a pick request?

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants