Fix (Performance): Improve performance for getting custom breakpoints

[mxkae]

fixes #3477

[github-actions]

🤖 Pull request artifacts

file	commit
pr3479-stackable-3479-merge.zip	438b8c9

[github-actions]

Try this Pull Request in the WordPress playground: https://playground.wordpress.net/?mode=seamless#{"landingPage":"/wp-admin/post-new.php?post_type=page","preferredVersions":{"php":"latest","wp":"latest"},"steps":[{"step":"login","username":"admin","password":"password"},{"step":"installPlugin","pluginZipFile":{"resource":"url","url":"https://raw.githubusercontent.com/gambitph/Stackable/artifacts/pr3479-stackable-3479-merge.zip"}}]}

[bfintal]

I feel that this implementation has a chance to break themes and other integrations that already successfully change the breakpoints, mainly because you are changing when the values are computed/loaded. Currently, the breakpoints are calculated when the css is about to be parsed, the new implementation does it early in the lifecycle, so existing hooks might not run correctly

Instead, I would recommend caching the values from within get_dynamic_breakpoints().

Use code profiler to check this, maybe this line is also slow:

return ! empty( $breakpoints['tablet'] ) || ! empty( $breakpoints['mobile'] );