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

Pin the floating-ui package versions in package-lock.json #4871

Conversation

tellthemachines
Copy link
Contributor

To avoid breaking updates.

Trac ticket: https://core.trac.wordpress.org/ticket/58847

Essentially copies the fix from the previous time we had this problem: #4155 (comment)

Hope we can make our popovers work with the latest versions of those packages before too much longer 😅 but given it's RC time again this seems like the safest option for now.


This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

Copy link
Member

@ramonjd ramonjd left a comment

Choose a reason for hiding this comment

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

LGTM

Thank you for fixing this! 🙇🏻

Before After

@huzaifaalmesbah
Copy link

huzaifaalmesbah commented Jul 20, 2023

Thank you very much for this solution. @tellthemachines

@tellthemachines
Copy link
Contributor Author

Committed to trunk in r56284. Pending testing on trunk to check the issue is fixed, we can then commit it to the release branch.

@audrasjb
Copy link
Contributor

@audrasjb audrasjb closed this Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

5 participants