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

Remove: Unrequired regex search on duotone supports. #42020

Merged
merged 1 commit into from
Jun 29, 2022

Conversation

jorgefilipecosta
Copy link
Member

This replaces a preg_replace replacement with a simple string replacement which is much more performant and is enough in this case.

@jorgefilipecosta jorgefilipecosta added [Type] Performance Related to performance efforts [Type] Code Quality Issues or PRs that relate to code quality labels Jun 28, 2022
@jorgefilipecosta jorgefilipecosta merged commit e776b4f into trunk Jun 29, 2022
@jorgefilipecosta jorgefilipecosta deleted the fix/remove-unrequired-regex-search branch June 29, 2022 09:31
@github-actions github-actions bot added this to the Gutenberg 13.6 milestone Jun 29, 2022
markjaquith pushed a commit to markjaquith/WordPress that referenced this pull request Sep 8, 2022
This add the ability to unset duotone in themes with default duotone set.

This commit backports the original PRs from Gutenberg repository:
* [WordPress/gutenberg#39564 #39564: Add ability to unset duotone in themes with default duotone set]
* [WordPress/gutenberg#42020 #42020: Remove: Unrequired regex search on duotone supports]

Follow-up to [50929], [52757].

Props ajlende, jorgefilipecosta, Joen, cbravobernal.
See #56467.
Built from https://develop.svn.wordpress.org/trunk@54101


git-svn-id: http://core.svn.wordpress.org/trunk@53660 1a063a9b-81f0-0310-95a4-ce76da25c4cd
github-actions bot pushed a commit to gilzow/wordpress-performance that referenced this pull request Sep 8, 2022
This add the ability to unset duotone in themes with default duotone set.

This commit backports the original PRs from Gutenberg repository:
* [WordPress/gutenberg#39564 #39564: Add ability to unset duotone in themes with default duotone set]
* [WordPress/gutenberg#42020 #42020: Remove: Unrequired regex search on duotone supports]

Follow-up to [50929], [52757].

Props ajlende, jorgefilipecosta, Joen, cbravobernal.
See #56467.
Built from https://develop.svn.wordpress.org/trunk@54101


git-svn-id: https://core.svn.wordpress.org/trunk@53660 1a063a9b-81f0-0310-95a4-ce76da25c4cd
pbearne pushed a commit to pbearne/wordpress-develop that referenced this pull request Sep 9, 2022
This add the ability to unset duotone in themes with default duotone set.

This commit backports the original PRs from Gutenberg repository:
* [WordPress/gutenberg#39564 #39564: Add ability to unset duotone in themes with default duotone set]
* [WordPress/gutenberg#42020 #42020: Remove: Unrequired regex search on duotone supports]

Follow-up to [50929], [52757].

Props ajlende, jorgefilipecosta, Joen, cbravobernal.
See #56467.

git-svn-id: https://develop.svn.wordpress.org/trunk@54101 602fd350-edb4-49c9-b593-d223f7449a82
whereiscodedude pushed a commit to whereiscodedude/wpss that referenced this pull request Sep 18, 2022
This add the ability to unset duotone in themes with default duotone set.

This commit backports the original PRs from Gutenberg repository:
* [WordPress/gutenberg#39564 #39564: Add ability to unset duotone in themes with default duotone set]
* [WordPress/gutenberg#42020 #42020: Remove: Unrequired regex search on duotone supports]

Follow-up to [50929], [52757].

Props ajlende, jorgefilipecosta, Joen, cbravobernal.
See #56467.
Built from https://develop.svn.wordpress.org/trunk@54101
ootwch pushed a commit to ootwch/wordpress-develop that referenced this pull request Nov 4, 2022
This add the ability to unset duotone in themes with default duotone set.

This commit backports the original PRs from Gutenberg repository:
* [WordPress/gutenberg#39564 #39564: Add ability to unset duotone in themes with default duotone set]
* [WordPress/gutenberg#42020 #42020: Remove: Unrequired regex search on duotone supports]

Follow-up to [50929], [52757].

Props ajlende, jorgefilipecosta, Joen, cbravobernal.
See #56467.

git-svn-id: https://develop.svn.wordpress.org/trunk@54101 602fd350-edb4-49c9-b593-d223f7449a82
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Code Quality Issues or PRs that relate to code quality [Type] Performance Related to performance efforts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant